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
python3Packages.papis: provide as a library too #59070
Conversation
Shouldn't |
fixed it thanks |
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.
Expressions of libraries belong under python-modules/
The project is turning into a dependency for several scripts and possibly UIs (see the different repositories at https://github.com/papis/) so it makes sense to have it as a library.
I was hesitant to move the files but I did it. |
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.
You need to add each Python dependency as an argument and you should specify disabled = !isPy3k
.
@@ -2,7 +2,7 @@ | |||
, python3, xdg_utils |
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.
, python3, xdg_utils | |
, xdg_utils |
@@ -2,7 +2,7 @@ | |||
, python3, xdg_utils | |||
}: | |||
|
|||
python3.pkgs.buildPythonApplication rec { | |||
python3.pkgs.buildPythonPackage rec { |
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.
python3.pkgs.buildPythonPackage rec { | |
buildPythonPackage rec { |
I've taken the review into account and added myself as maintainer. I have a good knowledge of the upcoming papis changes so I should do the next nix update. |
The project is turning into a dependency for several scripts and possibly UIs
(see the different repositories at https://github.com/papis/) so it makes sense
to have it as a library.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)