-
-
Notifications
You must be signed in to change notification settings - Fork 13.3k
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.packages.ghc865Binary: drop unevaluatable packages #278074
haskell.packages.ghc865Binary: drop unevaluatable packages #278074
Conversation
`haskell.packages.ghc865Binary` has a few packages that fails to evaluate by referring non-existent attributes. This turns evaluation attempts into unrecoverable errors. Before the change an attempt to instantiate all `ghc810` packages failed as: $ nix-instantiate --strict --eval --expr 'with import ./. {}; lib.mapAttrs (n: v: builtins.tryEval (lib.isDerivation v)) haskell.packages.ghc865Binary' error: … while evaluating the attribute 'llvmPackages' at lib/attrsets.nix:1180:11: 1179| attrs: 1180| attrs // { recurseForDerivations = false; }; | ^ 1181| error: value is null while a set was expected The change follows definition of `pkgs/top-level/haskell-packages.nix` and also sets `llvmPackages` to `null`. While at it dropped other references to deleted packages like `exceptions` pin.
Thx |
@@ -91,7 +95,7 @@ self: super: { | |||
ghc-lib-parser-ex = addBuildDepend self.ghc-lib-parser super.ghc-lib-parser-ex; | |||
|
|||
# This became a core library in ghc 8.10., so we don’t have an "exception" attribute anymore. | |||
exceptions = super.exceptions_0_10_4; | |||
exceptions = null; |
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.
We could still build a version of exceptions, but I suppose it hardly matters here.
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.
Good point! Let's restore it for consistency. Used 0.10.7
in proposed #278310
… version In NixOS#278074 I removed `exception` completely. But `nixpkgs` has a version of `exceptions we could use. Let's enable that instead.
… version In NixOS#278074 I removed `exception` completely. But `nixpkgs` has a version of `exceptions we could use. Let's enable that instead. Co-authored-by: Dennis Gosnell <cdep.illabout@gmail.com>
haskell.packages.ghc865Binary
has a few packages that fails to evaluate by referring non-existent attributes. This turns evaluation attempts into unrecoverable errors.Before the change an attempt to instantiate all
ghc810
packages failed as:The change follows definition of
pkgs/top-level/haskell-packages.nix
and also setsllvmPackages
tonull
. While at it dropped other references to deleted packages likeexceptions
pin.Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.