Skip to content

Add wasm cross compilation#658

Merged
3noch merged 7 commits into
reflex-frp:developfrom
dfordivam:wasm-support-1
May 1, 2020
Merged

Add wasm cross compilation#658
3noch merged 7 commits into
reflex-frp:developfrom
dfordivam:wasm-support-1

Conversation

@dfordivam

Copy link
Copy Markdown
Member

No description provided.

Comment thread haskell-overlays/reflex-packages/dep/reflex-dom/github.json Outdated
Comment thread default.nix Outdated
Comment thread nixpkgs/github.json
"branch": "release-19.09",
"rev": "49550f29cd9d6ed27f4a76ba3c3fe30edf882eb7",
"sha256": "003fpkrcc5mw33q670q4z865zpr430cllbxhyzj8k9p4hbc8yqda"
"branch": "wasm-r-p-1909-integ",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's this? can this get merged to a main branch?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some changes like (lib/systems/doubles.nix) which cannot be put in upstream, neither can they be done via overlay, so we will have to use a fork.

@Ericson2314 Ericson2314 May 1, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The lib/systems/doubles.nix change looks good to upstream to me. Did someone say otherwise?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get a PR up make a ticket to check back on it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ericson2314 yes, now I realized that it should be possible to upstream that, even though the cross-compilation to wasm will still happen by overlays.

Comment thread haskell-overlays/wasm/default.nix Outdated
@3noch 3noch merged commit 71e8726 into reflex-frp:develop May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants