Skip to content

Add dependencies to readme#97

Open
brndd wants to merge 3 commits into
baodrate:masterfrom
brndd:patch-1
Open

Add dependencies to readme#97
brndd wants to merge 3 commits into
baodrate:masterfrom
brndd:patch-1

Conversation

@brndd

@brndd brndd commented Jan 24, 2021

Copy link
Copy Markdown

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 :-)

Comment thread README.md Outdated
* awk
* util-linux
* sed
* openssh-clients

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isnt this optional?

Copy link
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Collaborator

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

Comment thread README.md
Comment thread README.md Outdated
### Optional dependencies:

* pv (for progress bar during backup)
* notify-send (for desktop notifications)

Copy link
Copy Markdown
Collaborator

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?

@wesbarnett wesbarnett Jan 30, 2021

Copy link
Copy Markdown
Collaborator

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.

Comment thread README.md
@wesbarnett

wesbarnett commented Jan 30, 2021

Copy link
Copy Markdown
Collaborator

Here's a list of programs called in the script:

  • mkfifo (coreutils package)
  • systemd-cat (systemd package)
  • which
  • notify-send (libnotify package) - optional
  • pv - optional
  • cat (coreutils package)
  • sudo - might not be optional?
  • findmnt (util-linux package)
  • grep
  • awk (gawk package)
  • btrfs
  • snapper
  • rsync - needed to send info.xml to remote hosts - optional
  • cp (coreutils package)
  • sync (coreutils package)
  • ssh (openssh package) - optional

So, at least on Arch Linux this is the list:

btrfs coreutils gawk grep snapper sudo systemd util-linux which

Excluding those in base this is:

btrfs snapper sudo which

And here's the optional list:

libnotify pv rsync openssh ssh

@NicoHood

Copy link
Copy Markdown
Collaborator

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?
By the way: You can check installed packages with command, see an example: https://github.com/NicoHood/gpgit/blob/master/gpgit.sh#L145

Aaaand you forgot the bash dependency ;-D

@wesbarnett

Copy link
Copy Markdown
Collaborator

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?

By the way: You can check installed packages with command, see an example: https://github.com/NicoHood/gpgit/blob/master/gpgit.sh#L145

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

@brndd

brndd commented Jan 31, 2021

Copy link
Copy Markdown
Author

OK, I've updated the list as you specified above. I included packages that on Arch would be included in base to make the list easier to adapt for people using other distros (and maybe some weirdos don't have base installed). btrfs send etc. seem to come from btrfs-progs on Arch too rather than btrfs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants