Skip to content

haskell.compiler.ghcjs: fix build with aeson 2.0 #168195

Merged
sternenseemann merged 2 commits intoNixOS:haskell-updatesfrom
sternenseemann:ghcjs-fix-build
Apr 12, 2022
Merged

haskell.compiler.ghcjs: fix build with aeson 2.0 #168195
sternenseemann merged 2 commits intoNixOS:haskell-updatesfrom
sternenseemann:ghcjs-fix-build

Conversation

@sternenseemann
Copy link
Member

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.05 Release Notes (or backporting 21.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Apr 10, 2022
@sternenseemann

This comment was marked as outdated.

@sternenseemann sternenseemann changed the title Ghcjs fix build haskell.compiler.ghcjs: fix build with aeson 2.0 Apr 10, 2022
@sternenseemann sternenseemann linked an issue Apr 10, 2022 that may be closed by this pull request
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Apr 10, 2022
By telling Hydra where to look explicitly, it can be instrumented
to build the boot ghcjs which is only about 300MB. This saves at
least some time compiling for users and will also provide some
eval CI automatically.
@sternenseemann
Copy link
Member Author

@ofborg build haskell.compiler.ghcjs.bootGhcjs

@cdepillabout
Copy link
Member

cc @dfordivam

Copy link
Contributor

@dfordivam dfordivam left a comment

Choose a reason for hiding this comment

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

I believe the webdriver changes breaks the compilation with aeson <2.0. But for nixpkgs that is not an issue (I think), so approving.

@sternenseemann
Copy link
Member Author

I believe the webdriver changes breaks the compilation with aeson <2.0

Indeed, that is why I've only applied it to the bootstrap ghcjs package set for now.

@sternenseemann sternenseemann merged commit 51131f7 into NixOS:haskell-updates Apr 12, 2022
@sternenseemann sternenseemann deleted the ghcjs-fix-build branch April 12, 2022 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: haskell General-purpose, statically typed, purely functional programming language 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

haskell.compiler.ghcjs has a broken dependency

3 participants