-
Notifications
You must be signed in to change notification settings - Fork 247
nixpkgs 20.09 pkg-config has a prefix #874
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
Conversation
From nixpkgs 20.09, the pkg-config exe has a prefix matching the ghc one
From nixpkgs 20.09, the pkg-config exe has a prefix matching the ghc one
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.
Seems fine. I haven't really been following the way this stuff works in nixpkgs: is it really expected that every package manually prefixes these tools with the target prefix? That seems annoying...
builder/comp-builder.nix
Outdated
@@ -91,12 +91,18 @@ let | |||
"$(cat ${configFiles}/configure-flags)" | |||
] ++ commonConfigureFlags); | |||
|
|||
# From nixpkgs 20.09, the pkg-config exe has a prefix matching the ghc one | |||
pkgConfigHasPrefix = !(lib.strings.hasPrefix "19." lib.nixpkgsVersion | |||
|| lib.strings.hasPrefix "20.03" lib.nixpkgsVersion); |
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.
Can't we do one of those version comparison thingys we do elsewhere, e.g. versionLessThan "20.09"
?
Apparently compareVersions treats the string "pre" in version differently. For details see https://nixos.org/manual/nix/stable/#ssec-version-comparisons
This will include the patch that was added in input-output-hk/haskell.nix#874 to fix this issue: ``` Setup: The program 'pkg-config' version >=0.9.0 is required but it could not be found. builder for '/nix/store/16bf6j2f1fig368pi1yfv70x5njs8d6y-cardano-crypto-class-lib-cardano-crypto-class-2.0.0-x86_64-w64-mingw32.drv' failed with exit code 1 ```
This will include the patch that was added in input-output-hk/haskell.nix#874 to fix this issue: ``` Setup: The program 'pkg-config' version >=0.9.0 is required but it could not be found. builder for '/nix/store/16bf6j2f1fig368pi1yfv70x5njs8d6y-cardano-crypto-class-lib-cardano-crypto-class-2.0.0-x86_64-w64-mingw32.drv' failed with exit code 1 ```
This will include the patch that was added in input-output-hk/haskell.nix#874 to fix this issue: ``` Setup: The program 'pkg-config' version >=0.9.0 is required but it could not be found. builder for '/nix/store/16bf6j2f1fig368pi1yfv70x5njs8d6y-cardano-crypto-class-lib-cardano-crypto-class-2.0.0-x86_64-w64-mingw32.drv' failed with exit code 1 ```
This will include the patch that was added in input-output-hk/haskell.nix#874 to fix this issue: ``` Setup: The program 'pkg-config' version >=0.9.0 is required but it could not be found. builder for '/nix/store/16bf6j2f1fig368pi1yfv70x5njs8d6y-cardano-crypto-class-lib-cardano-crypto-class-2.0.0-x86_64-w64-mingw32.drv' failed with exit code 1 ```
From nixpkgs 20.09, the pkg-config exe has a prefix matching the ghc one
From nixpkgs 20.09, the pkg-config exe has a prefix matching the ghc one