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

Ghcjs 8.2 & 8.4 #40392

Merged
merged 11 commits into from May 17, 2018
Merged

Ghcjs 8.2 & 8.4 #40392

merged 11 commits into from May 17, 2018

Conversation

ElvishJerricco
Copy link
Contributor

@ElvishJerricco ElvishJerricco commented May 12, 2018

Motivation for this change

Adds GHCJS 8.2 and GHCJS 8.4.

These builds are currently blocked on ghcjs/ghcjs#654. It's important that these builds be checked more than once until that bug is fixed, as it is an intermittent failure.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@ElvishJerricco
Copy link
Contributor Author

Seems rebasing caused the build to fail.

@ElvishJerricco
Copy link
Contributor Author

ElvishJerricco commented May 12, 2018

Ah, no I just had a dirty work tree. Builds fine! Er, apparently not... There must be an intermittent error.

@ElvishJerricco ElvishJerricco changed the title WIP: Ghcjs 8.2 Ghcjs 8.2 May 12, 2018
@ElvishJerricco ElvishJerricco changed the title Ghcjs 8.2 WIP: Ghcjs 8.2 May 12, 2018
@ElvishJerricco ElvishJerricco requested a review from peti as a code owner May 12, 2018 21:11
@ElvishJerricco ElvishJerricco changed the title WIP: Ghcjs 8.2 WIP: Ghcjs 8.2 & 8.4 May 12, 2018
@ElvishJerricco
Copy link
Contributor Author

Adding GHCJS 8.4 as well, since they're closely related and both have build failures

@ElvishJerricco ElvishJerricco changed the title WIP: Ghcjs 8.2 & 8.4 Ghcjs 8.2 & 8.4 May 17, 2018
@ElvishJerricco
Copy link
Contributor Author

Ok we have what seems to be a working workaround for the panic. So I think this is good to merge. @peti what do you think?

buildInputs = [cabal2nix yq];
} ''
(
printf '{ callPackage, configuredSrc }:\n\n{\n\n'
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be worth a comment explaining what this is doing O_O

@peti peti merged commit b5cef2e into NixOS:master May 17, 2018
@ElvishJerricco
Copy link
Contributor Author

@peti Is there any change necessary to have these built by hydra and cached? Or will it do the compilers automatically?

@peti
Copy link
Member

peti commented May 17, 2018

Yes, hydra.nixos.org will compile them; no further changes necessary.

@ElvishJerricco
Copy link
Contributor Author

@peti are you sure? The expressions are on the nixos-unstable channel but they are not cached.

@peti
Copy link
Member

peti commented May 21, 2018

They haven't compiled on Hydra successfully yet: https://hydra.nixos.org/eval/1458249?filter=ghcjs&compare=1457965

@peti
Copy link
Member

peti commented Jun 3, 2018

@ElvishJerricco, neither of these builds has ever succeeded on Hydra:

Can you please take a look at these issues? If the package does not build, then we should have this code in Nixpkgs and I'm inclined to revert these changes if that state does not change.

@ElvishJerricco
Copy link
Contributor Author

ElvishJerricco commented Jun 3, 2018

Right, sorry. Had to wait on the GHC 8.0 fix. Though that build error is super obscure. The log is a binary?? Nope, just looks that way in Safari for me. Firefox displays a normal log.

@3noch
Copy link
Contributor

3noch commented Jun 7, 2018

This is now in nixpkgs-unstable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants