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
yubikey-agent: init at 0.1.3 #92965
yubikey-agent: init at 0.1.3 #92965
Conversation
27e89ed
to
3a69249
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/pcsc-error-compiling-yubikey-agent/8084/1 |
It seems https://github.com/go-piv/piv-go doesn't properly use pkgconfig to ask for the include path, but instead hardcodes CFLAGS. They even have a TODO in their code: https://github.com/go-piv/piv-go/blob/master/piv/pcsc_unix.go#L21-L31 If we discover things with pkgconfig, the right paths are discovered:
|
@flokli Does that mean there's nothing I can do to make this work, as it stands? |
Of course :-) It should just be enough to replace L21-L29 with a
I propose sending a patch upstream to https://github.com/go-piv/piv-go, and then applying it locally in the yubikey-agent derivation, until it's merged and trickled downstream. |
Thank you for your help, @flokli 👍 |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/another-missing-header-question/7119/3 |
Upstream PR: go-piv/piv-go#75 |
I think #92936 wants to package the same thing, right? |
yes, indeed, and #92936 contains a temporary patch to workaround the missing pkgconfig directive PRed in go-piv/piv-go#75. Let's close this one. |
Sure, happy to help, @philandstuff |
This PR is failing, if anyone can help; I'd really appreciate it.
I tried patching, but permissions fail, which I guess it because it's an input.
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)