-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
mackup: init at 0.8.16 #35163
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
mackup: init at 0.8.16 #35163
Conversation
python2Packages.buildPythonApplication rec { | ||
name = "mackup-0.8.16"; | ||
|
||
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
sha256 = "0f5cb5srxa3bn7lbfs5rb4iqjs57blzvgypivndy756vsvm9ijrv"; | ||
}; | ||
|
||
"docopt" = python2Packages.buildPythonPackage { |
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.
Please put this in pkgs/development/python-modules
and use callPackage
in python-packages.nix
.
"docopt" = python2Packages.buildPythonPackage { | ||
name = "docopt-0.6.2"; | ||
doCheck = false; | ||
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
|
||
"docopt" = python2Packages.buildPythonPackage { | ||
name = "docopt-0.6.2"; | ||
doCheck = false; |
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.
Why?
}; | ||
|
||
"docopt" = python2Packages.buildPythonPackage { | ||
name = "docopt-0.6.2"; |
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.
pname
and version
instead
}: | ||
|
||
python2Packages.buildPythonApplication rec { | ||
name = "mackup-0.8.16"; |
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.
pname
and version
instead
{ lib | ||
, fetchurl | ||
, python2Packages | ||
, stdenv |
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.
stdenv
not needed, use lib
@@ -3431,6 +3431,8 @@ with pkgs; | |||
|
|||
macchanger = callPackage ../os-specific/linux/macchanger { }; | |||
|
|||
mackup = callPackage ../tools/backup/mackup { }; |
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 can use python2Packages.callPackage
and specify the dependencies as arguments instead of python2Packages
.
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.
Hey, thanks for the feedback. I'll try to get to this in the next day or two.
Any progress on this? |
Motivation for this change
New application mackup for handling dot files.
Things done
- [ ] Tested using sandboxing (nix.useSandbox on NixOS, or option(I think this is broken on darwin at the moment)build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)