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
rdbtools: init at 0.1.14 #68631
rdbtools: init at 0.1.14 #68631
Conversation
6841b55
to
1cd69a9
Compare
Fixed eval error, should be good now |
1cd69a9
to
929f6c1
Compare
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.
probably future proofing, but only referencing the python interpreter allows better flexibility with overrides
108ebef
to
3f6cfe8
Compare
Ok, i implemented suggested changes. Also using default version of |
i would highly recommend python3, python2 is EOL in 2 weeks |
Ok, but works both with both versions, so when we change the default version to |
3f6cfe8
to
df0ff77
Compare
Ok, i set |
It would be the same thing with python2, the python closure in nixpkgs is ~110MB, and it get's "reused" by many other packages, so it's actual cost is quite amortized. |
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.
diff LGTM
commits LGTM
[7 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/68631
4 package were built:
python27Packages.python-lzf python37Packages.python-lzf python38Packages.python-lzf rdbtools
@GrahamcOfBorg build python27Packages.python-lzf python37Packages.python-lzf python38Packages.python-lzf rdbtools |
Motivation for this change
Package rdbtools
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)Notify maintainers
cc @