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
p11-kit: Add libtasn1 to nativeBuildInputs #96868
Conversation
@GrahamcOfBorg build p11-kit |
do you mind rebasing on top of the latest master? You checked out a commit in the 10mins in which I accidentally broke it :( specifically:
|
This PR is actually targeted against |
oh sorry, do you mind rebasing on the latest target branch? xD |
Not at all, but someone would have to run the whole |
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.
diff LGTM
I don't think this will cause any issues
For cross platform builds, p11-kit, libtasn1 in nativeBuildInputs provides the asn1Parser binary on the hostPlatform needed for building. At the same time, libtasn1 in buildInputs provides the libasn1 library to link against for the target platform. Hence, libtasn1 is required in both native and build inputs.
Motivation for this change
For cross platform builds of p11-kit, libtasn1 in nativeBuildInputs
provides the asn1Parser binary on the hostPlatform needed for building.
At the same time, libtasn1 in buildInputs provides the libasn1 library
to link against for the target platform.
Hence, libtasn1 is required in both native and build inputs.
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)