-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add dependencies to readme #97
base: master
Are you sure you want to change the base?
Conversation
README.md
Outdated
* awk | ||
* util-linux | ||
* sed | ||
* openssh-clients |
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.
Isnt this optional?
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.
The only one of these four required is awk
.
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 :-)