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
s3ql: clean up dependencies, build for release, fix tests, fix --systemd
option, fix umount.s3ql
#67541
Conversation
--systemd
option
--systemd
option--systemd
option, fix umount.s3ql
Thank you for your contributions.
|
psmisc # `umount.s3ql` requires `fuser` | ||
utillinuxMinimal.bin # `umount.s3ql` requires `umount` |
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.
This will not put those dependencies on PATH
. You will need to patch the paths in https://github.com/s3ql/s3ql/blob/02b4b5b0775e3e9fb61581d4ab917aa351802b43/src/s3ql/umount.py.
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.
Looking at my logs from the time I made this patch, umount.s3ql
gave errors like FileNotFoundError: [Errno 2] No such file or directory: 'fuser': 'fuser'
. Adding those packages to buildInputs
did not fix the errors, but adding them to propagatedBuildInputs
did. Someone who understands the role of propagatedBuildInputs
better than myself may have better insight.
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 use makeWrapper instead of abusing propagatedBuildInputs.
Clean up dependencies, build for release, fix tests, fix `--systemd` option, and fix `umount.s3ql`.
nativeBuildInputs = with python3Packages; [ | ||
cython | ||
]; | ||
buildInputs = [ | ||
sqlite | ||
]; | ||
checkInputs = with python3Packages; [ | ||
pytest | ||
] ++ [ | ||
which | ||
]; |
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.
nativeBuildInputs = with python3Packages; [ | |
cython | |
]; | |
buildInputs = [ | |
sqlite | |
]; | |
checkInputs = with python3Packages; [ | |
pytest | |
] ++ [ | |
which | |
]; | |
nativeBuildInputs = [ python3Packages.cython ]; | |
buildInputs = [ sqlite ]; | |
checkInputs = [ python3Packages.pytest which ]; |
psmisc # `umount.s3ql` requires `fuser` | ||
utillinuxMinimal.bin # `umount.s3ql` requires `umount` |
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 use makeWrapper instead of abusing propagatedBuildInputs.
I marked this as stale due to inactivity. → More info |
I haven't used s3ql in a while. If anyone else wants to re-open this issue with the requested changes, feel free to use my code changes. |
Motivation for this change
Move dependencies to proper inputs. Build for release instead of development. Fix and enable all tests. Use the correct
crypography
library instead ofpycrypto
, as per installation instructions.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 @rushmorem