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
treewide: pkgs.pkgconfig -> pkgs.pkg-config, move pkgconfig to alias.nix #109887
Conversation
I'm going to ignore the white space check, because I value my sanity when editing 1717 files |
whitespace PR: #109911 |
196cf4d
to
e46f82d
Compare
last two failures are from from not meeting
|
continuation of NixOS#109595 pkgconfig was aliased in 2018, however, it remained in all-packages.nix due to its wide usage. This cleans up the remaining references to pkgs.pkgsconfig and moves the entry to aliases.nix. python3Packages.pkgconfig remained unchanged because it's the canonical name of the upstream package on pypi.
1cf2a37
to
2befb3c
Compare
this is annoying, running ofborg locally with |
once ofborg passes, I'm merging |
alright, I think this is in good enough shape. Merging as I want to go to bed, and need to wake up in ~5 hours |
I hope I do not merge a PR which introduces new build failures by using old ofborg evals. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/pkgconfig-alias-has-been-officially-aliased/11066/1 |
Motivation for this change
continuation of #109595
pkgconfig
was aliased in 2018, however, it remained inall-packages.nix due to its wide usage. This cleans
up the remaining references to pkgs.pkgsconfig and
moves the entry to aliases.nix.
python3Packages.pkgconfig remained unchanged because
it's the canonical name of the upstream package
on pypi.
cc @siraben @SuperSandro2000
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)