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
haskell.compiler.ghcjs86: Bump ghc-8.6 branch, fix build. #68989
Conversation
74d6877
to
8e6e631
Compare
1e069cd
to
b587286
Compare
7d924d6
to
502976a
Compare
src = configuredSrc + /lib/ghc-api-ghcjs; | ||
libraryHaskellDepends = [ | ||
array base binary bytestring containers deepseq directory filepath | ||
ghc-boot ghc-boot-th ghc-heap ghci-ghcjs hpc process | ||
template-haskell-ghcjs terminfo time transformers unix | ||
]; | ||
libraryToolDepends = [ alex happy ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you change this manually, or did genStage0
do it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file comes from genStage0
.
@@ -1,7 +1,11 @@ | |||
{ haskellLib }: | |||
{ haskellLib, happy_1_19_5 }: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't you just use self.happy_1_19_5
rather than passing it in this way? Why is this version needed anyway?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ghc-api-ghcjs has the constraint happy >= 1.19 && <= 1.19.9
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't get it from self
because common-overrides.nix
didn't use self
.
d075238
to
742348d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not edit hackage-packages.nix
.
502976a
to
d6624a2
Compare
d6624a2
to
003ccae
Compare
@peti This PR is not meant to edit |
Oh, you are right. I hadn't realized that! |
@peti Thanks! What's required to get ghcjs86 building on Hydra again? |
Hydra builds all non-broken attributes in |
I can't seem to find it in the trunk jobset or the haskell-updates jobset, but maybe I'm looking in the wrong place? |
Does |
Does meta.hydraPlatforms = [] here prevent it from being built?
Yes.
|
This fixes the GHCJS 8.6 build. Previously the build was failing because the utils/makePackages.sh script was booting the GHC source tree before copying in the lib/* extra packages. This caused cabal to fail, since those packages are mentioned in GHCJS' cabal.project file. Bumping the ghc-8.6 branch fixed this.
ghc-api-ghcjs requires happy >= 1.19 && < 1.19.12, so happy 1.19.5 is used.
Tested on macOS, will test on Linux.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)