Skip to content

Fix building and testing with the JS backend #290

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

Merged
merged 1 commit into from
Jul 23, 2023

Conversation

hsyl20
Copy link
Contributor

@hsyl20 hsyl20 commented Jul 19, 2023

Note that most tests have been disabled because they don't pass due to missing C function support (e.g. pipe). But it's a good basis to start adding tests for supported things (e.g. touchFd, cf #285).

This requires a recent version of GHC that provides getMonotonicTimeNSec. See
https://gitlab.haskell.org/ghc/ghc/-/issues/23687 and https://gitlab.haskell.org/ghc/ghc/-/merge_requests/10900.

Note that most tests have been disabled because they don't pass due
to missing C function support (e.g. pipe). But it's a good basis to
start adding tests for supported things (e.g. touchFd, cf haskell#285).

This requires a recent version of GHC that provides
getMonotonicTimeNSec. See
https://gitlab.haskell.org/ghc/ghc/-/issues/23687 and
https://gitlab.haskell.org/ghc/ghc/-/merge_requests/10900.
@hsyl20
Copy link
Contributor Author

hsyl20 commented Jul 19, 2023

For some reason, Fedora mirror site is 404

@hsyl20 hsyl20 mentioned this pull request Jul 19, 2023
@bgamari
Copy link
Contributor

bgamari commented Jul 19, 2023

Thanks for doing this, @hsyl20 !

@Bodigrim
Copy link
Contributor

@hsyl20 is it expected that this PR submits config.{guess,sub}?

@hsyl20
Copy link
Contributor Author

hsyl20 commented Jul 23, 2023

Yes otherwise the platform isn't recognized by the configure script.

@hsyl20
Copy link
Contributor Author

hsyl20 commented Jul 23, 2023

(we overwrite these files when we build the package in the ghc tree)

@hasufell hasufell self-requested a review July 23, 2023 14:43
@hasufell hasufell merged commit 48d590c into haskell:master Jul 23, 2023
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.

4 participants