snap-sync icon indicating copy to clipboard operation
snap-sync copied to clipboard

Add dependencies to readme

Open brndd opened this issue 5 years ago • 4 comments

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

brndd avatar Jan 24 '21 14:01 brndd

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

wesbarnett avatar Jan 30 '21 02:01 wesbarnett

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

NicoHood avatar Jan 30 '21 08:01 NicoHood

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

wesbarnett avatar Jan 30 '21 12:01 wesbarnett

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.

brndd avatar Jan 31 '21 16:01 brndd