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
pythonPackages.dotfiles: 0.6.3 -> 0.6.4 #29066
Conversation
Thanks. This is an application, and not a Python library. Could you move the expression out of |
b18a7a3
to
09051b0
Compare
Ok - moved 'dotfiles' out of pythonPackages as requested. Kept the version update as separate commit. |
bump |
@@ -0,0 +1,23 @@ | |||
{ stdenv, fetchurl, python27Packages }: |
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.
Is explicitly python27 needed, or could it also potentially work with other versions of Python? We try to stay as generic as possible, so pythonPackages
has preference, but if it doesn't work with 3.x, then python2Packages
(or python27Packages
, that doesn't really matter anymore) should be used.
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.
license = licenses.isc; | ||
}; | ||
|
||
src = fetchurl { |
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.
fetchPypi
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.
sha256 = "03qis6m9r2qh00sqbgwsm883s4bj1ibwpgk86yh4l235mdw8jywv"; | ||
}; | ||
|
||
buildInputs = with python27Packages; [ pytest ]; |
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.
checkInputs
(note that his only a Python thing)
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.
09051b0
to
6a4be4a
Compare
Thanks. Pushed e1385f0a9461a1112c74f22f37ed3d56b5620770 and also to 17.09. |
Thanks! FYI - I don't yet see e1385f0a9, though I see the cherry-picked commit in 17.09 |
Oh, forgot to push it. So, its d6cebc8 now. |
Motivation for this change
upgrade minor version
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)