Conversation
README.md
Outdated
| * awk | ||
| * util-linux | ||
| * sed | ||
| * openssh-clients |
There was a problem hiding this comment.
The only one of these four required is awk.
There was a problem hiding this comment.
actually util-linux is required for findmnt
README.md
Outdated
| ### Optional dependencies: | ||
|
|
||
| * pv (for progress bar during backup) | ||
| * notify-send (for desktop notifications) |
There was a problem hiding this comment.
Will the app crash, if you dont have notify-send installed, and do not specify --quiet? Looking at the code, it will crash, maybe that should be fixed?
There was a problem hiding this comment.
libnotify is the optional dependency (which provides notify-send). The app won't crash if it is not present. It will continue as if the user specified --quiet.
|
Here's a list of programs called in the script:
So, at least on Arch Linux this is the list: Excluding those in And here's the optional list: |
|
Why is sudo required? Cant you just su as root and then do the operation? I mean I never did that for some pretty obvious reasons, but it should be possible, right? Which should be installed on every system, right? Aaaand you forgot the bash dependency ;-D |
sudo is needed for notifications so it would still be optional https://github.com/wesbarnett/snap-sync/blob/276e17794064ea8f26c4e28a6c5672b5ca00fadb/bin/snap-sync#L163 |
|
OK, I've updated the list as you specified above. I included packages that on Arch would be included in |
This is probably useful information to have.
The branch is open to edits from maintainers so feel free to change things if it's not formatted like you prefer, or just close the PR and do it manually on the main repo :-)