borgbackup: cleanup inputs
tox/detox are removed; they are used to test multiple python
implementations in parallel, which isn't really appropriate for a Nix
install check. Also, because these were added to the propagated build
inputs, lots of unnecessary stuff ended up in the resulting closure.
Note that removing tox does not imply tests are not run; they never were
to begin with; this is a separate issue.
This fixes https://github.com/NixOS/nixpkgs/issues/16171
Also, for "correctness" ordinary build-time dependencies are added to
`buildInputs`; only inputs that are to be added to the python path are
added to `propagatedBuildInputs`.
commit5195e0d70fd648e4d3e2c33dbe95537b4b480b81
authorJoachim Fasting <joachifm@fastmail.fm>
dateSat 31 Dec 2016, 20:44 (10 years ago)
parent61027b7
pkgs/tools/backup/borg/default.nix+6−5
@@ -15,11 +15,12 @@ python3Packages.buildPythonApplication rec {
1515 # For building documentation:
1616 sphinx sphinx_rtd_theme
1717 ];
18− propagatedBuildInputs = [
19− acl lz4 openssl
20− ] ++ (with python3Packages; [
21− cython msgpack llfuse tox detox setuptools_scm
22− ]);
18+ buildInputs = [
19+ acl lz4 openssl python3Packages.setuptools_scm
20+ ];
21+ propagatedBuildInputs = with python3Packages; [
22+ cython llfuse msgpack
23+ ];
2324
2425 preConfigure = ''
2526 export BORG_OPENSSL_PREFIX="${openssl.dev}"