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
hslib: Function to extract the haskell build inputs of a package. #33012
Conversation
This works by extracting out some logic from generic-builder.nix to make it possible to get at the relevant information by overriding mkDerivation for the haskell package.
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.
Just incude the native ones too and it's good with me.
allPkgconfigDepends = pkgconfigDepends ++ libraryPkgconfigDepends ++ executablePkgconfigDepends ++ | ||
optionals doCheck testPkgconfigDepends ++ optionals doBenchmark benchmarkPkgconfigDepends; | ||
inherit (haskellLib.extractBuildInputs ghc (args // { inherit doCheck doBenchmark; })) | ||
allPkgconfigDepends propagatedBuildInputs otherBuildInputs haskellBuildInputs systemBuildInputs; | ||
|
||
nativeBuildInputs = optional (allPkgconfigDepends != []) pkgconfig ++ |
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.
Don't forget these too!
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.
Nothing has changed with these, they're not available in pkg.env either...
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.
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.
I just mean that the definition of nativeBuildInputs hasn't changed nor has its usage. Nothing in this PR required extracting it out. There's no reason not to do so in the future if there's a need, but there's also no reason to do so now...
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.
That's fine then.
Reverted in 65138e8. Please give a chance to review before merging changes to the Haskell generic builder. |
@peti this was open for 5 days and had a positive review from another significant contributor with no comments from you. Was there any problem with this? |
Note that this doesn't change behavior at all. |
This works by extracting out some logic from generic-builder.nix to
make it possible to get at the relevant information by overriding
mkDerivation for the haskell package.