-
-
Notifications
You must be signed in to change notification settings - Fork 15.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
haskellPackages.bank-holidays-england: unbroken #84001
Conversation
@GrahamcOfBorg eval |
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.
while evaluating the attribute 'buildInputs' of the derivation 'matterhorn-50200.8.0' at /home/jon/.cache/nixpkgs-review/pr-84001/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:291:3:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'brick-skylighting-0.3' at /home/jon/.cache/nixpkgs-review/pr-84001/nixpkgs/pkgs/development/haskell-modules/generic-builder.nix:291:3:
while evaluating 'getOutput' at /home/jon/.cache/nixpkgs-review/pr-84001/nixpkgs/lib/attrsets.nix:464:23, called from undefined position:
while evaluating anonymous function at /home/jon/.cache/nixpkgs-review/pr-84001/nixpkgs/pkgs/stdenv/generic/make-derivation.nix:156:17, called from undefined position:
attribute 'brick_0_52_1' missing, at /home/jon/.cache/nixpkgs-review/pr-84001/nixpkgs/pkgs/development/haskell-modules/configuration-common.nix:1472:15
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.
to get it to build:
$ git diff
diff --git a/pkgs/development/haskell-modules/configuration-common.nix b/pkgs/development/haskell-modules/configuration-common.nix
index e34126053df..639329a22ea 100644
--- a/pkgs/development/haskell-modules/configuration-common.nix
+++ b/pkgs/development/haskell-modules/configuration-common.nix
@@ -1469,7 +1469,7 @@ self: super: {
# 2020-03-31 - "..." it broke again. so here's a more complicated way -- kiwi
matterhorn = super.matterhorn.override {
brick-skylighting = self.brick-skylighting.override {
- brick = self.brick_0_52_1.override {
+ brick = self.brick.override {
vty = self.vty_5_28_2;
};
};
diff --git a/pkgs/development/haskell-modules/hackage-packages.nix b/pkgs/development/haskell-modules/hackage-packages.nix
index bdb222b6c99..c906b017858 100644
--- a/pkgs/development/haskell-modules/hackage-packages.nix
+++ b/pkgs/development/haskell-modules/hackage-packages.nix
@@ -36239,7 +36239,6 @@ self: {
description = "Calculation of bank holidays in England and Wales";
license = stdenv.lib.licenses.bsd3;
hydraPlatforms = stdenv.lib.platforms.none;
- broken = true;
}) {};
"banwords" = callPackage
also, please target master when doing this
haskell-updates is for @peti to use when doing large rebuilds from hackage
Sorry, I was following the instructions from here: https://discourse.nixos.org/t/call-to-action-for-updating-haskell-packages-after-bump-to-lts-15/6071
|
I will defer to @cdepillabout about this. I think the hackage2nix needs to be re-ran to propagate the changes. |
I've used this branch is the last couple of fixes I've submitted. This is fine. |
This looks good to me, but will probably be superseded by https://github.com/NixOS/nixpkgs/pull/84001/files. |
@cdepillabout that link is back to this PR, is there is a different PR you meant to link to? |
Closing in favour of #84039 |
Motivation for this change
haskellPackages.bank-holidays-england
is not broken.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)