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
python: add use-pkgs-prefix option to update script #83334
Conversation
cc @FRidh |
I use this also for applications that are not part of the package set. |
82b1f01
to
b350851
Compare
b350851
to
5143c24
Compare
added a opt-in switch, we may later want to make this into a switch for applications where they wouldn't have a prefix (E.g. ansible). But, current usage looks like:
|
5143c24
to
2088406
Compare
updated commit msg and title to reflect addition of option |
Have not tested it, but I suppose the following still works?
|
2088406
to
b6aacae
Compare
|
As part of my "cleanup", I changed |
cc @FRidh I think this is good to go :) |
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.
maybe make it python3Packages? Hopefully more people start writing that, easier copy-pasting when invoking ofborg.
b6aacae
to
dfb6933
Compare
agreed about python3Packages. It's the better default |
Motivation for this change
before the colon should specify the installable path
python: pname: ...
->pythonPackages.pname: ...
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)