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
gitAndTools.pass-git-helper: init at 0.4 #47056
Conversation
{ stdenv, buildPythonPackage, isPy3k, fetchFromGitHub, pyxdg }: | ||
|
||
buildPythonPackage rec { | ||
pname = "git-pass-helper"; |
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.
We put most git related packages into pkgs/applications/version-management/git-and-tools
. Would this one fit there too?
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.
I guess so. Moved.
6f0c5ec
to
732b273
Compare
@@ -0,0 +1,24 @@ | |||
{ stdenv, buildPythonPackage, isPy3k, fetchFromGitHub, pyxdg }: | |||
|
|||
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.
okay. Now we have to decide if it is a python package that you want to use a dependency or an application where we use the executable from. This seems to be the latter one. In this case use buildPythonApplication
and reference it in pkgs/applications/version-management/git-and-tools/default.nix
. It will be then exposed as gitAndTools.pass-git-helper
.
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.
Should I delete the reference from /pkgs/top-level/python-packages.nix
then?
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.
yes.
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.
Done.
{ stdenv, buildPythonPackage, isPy3k, fetchFromGitHub, pyxdg }: | ||
|
||
buildPythonPackage rec { | ||
pname = "git-pass-helper"; |
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.
pass-git-helper instead of git-pass-helper?
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.
Sure, you are right.
732b273
to
45dc266
Compare
45dc266
to
3c08fd0
Compare
Motivation for this change
Adding pass integration for git credentials.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)