Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update GHCJS conditions to also support the GHC JavaScript backend #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JoshMeredith
Copy link

This MR is to update CPP and cabal file conditions to support the GHC JavaScript backend.

Because the JavaScript backend uses a different FFI syntax than GHCJS, the initialSeedJS function now has two versions of the import string.

The main motivation for this change is the use of this function in ghcjs-base's testsuite.

@phadej
Copy link
Collaborator

phadej commented Jun 21, 2023

My opinion is same as in #73

Add a CI job to simple.yml. I'll merge faster if there is a bindist installable with ghcup or from apt repository.

The #73 is not merged, because we apparently need some GHC specifc WASI toolchain, and I'm not gonna bother with something like that. I cannot easily debug if there is a problem, I don't care about accepting the patch (and taking maintenance burden).

@hasufell
Copy link

This patch doesn't appear to be in head.hackage?

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.

None yet

3 participants