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
haskellPackages.base-compat-batteries: fix bad dependency #39456
Conversation
# cabal2nix generates a dependency on base-compat, which is the wrong version | ||
base-compat-batteries = overrideCabal super.base-compat-batteries (drv: { | ||
libraryHaskellDepends = | ||
with pkgs.haskellPackages; [ base base-compat_0_10_1 ]; |
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.
The proper way to fix that issue is to pass the appropriate version of the package as an argument to the build expression, i.e. by using override
or overrideScope
(if necessary). If you look around in that file, you'll find plenty of examples how to do that.
8778132
to
c8b9ac9
Compare
Changed and pushed, thank you. I think I've done the right thing now. I can't see an obvious order to the packages that get tweaked. Should there be one? |
@GrahamcOfBorg build haskellPackages.base-compat-batteries @endgame, no, the order in that file does not matter much. |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: haskellPackages.base-compat-batteries Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: haskellPackages.base-compat-batteries Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: haskellPackages.base-compat-batteries Partial log (click to expand)
|
Motivation for this change
cabal2nix
generates an incorrect dependency onhaskellPackages.base-compat
, which is still at 0.9. This may not be the correct way to fix it.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-build -A haskellPackages.base-compat-batteries
nix-shell -p nox --run "nox-review wip"
./result/bin/
)