Add Rclone backup type

[cadenkriese]

Implemented Rclone as a backup type. This is primarily a derivative of the tar backup method but also uses code from #8.

See README.md for documentation.

[itzg]

Sorry for the delay reviewing this. Just the typo/extra variable I commented about and then this is good to merge.

[itzg]

Should this be

Suggested change

	: "${RCLONE_REPOSITORY:={DEST_DIR}"
	: "${RCLONE_REPOSITORY:=${DEST_DIR}}"

...however it also looks like that variable is not used or documented.

[cadenkriese]

Sorry, not very fluent in bash yet. I'm trying to make the RCLONE_REPOSITORY default to /backups, or if the user used the DEST_DIR variable instead of RCLONE_REPOSITORY, then it would still work as they expected.

[itzg]

...ah, given https://github.com/itzg/docker-mc-backup/pull/51/files#r774779939 it seems like you'll want this to be

Suggested change

	command rclone lsf --format "tp" "${RCLONE_REMOTE}:${DEST_DIR}" | grep ${BACKUP_NAME} | awk \
	-v DESTINATION="${DEST_DIR}" \
	command rclone lsf --format "tp" "${RCLONE_REMOTE}:${RCLONE_REPOSITORY}" | grep ${BACKUP_NAME} | awk \
	-v DESTINATION="${RCLONE_REPOSITORY}" \

The way to read ${RCLONE_REPOSITORY:=${DEST_DIR}} is "If RCLONE_REPOSITORY isn't already set, set it to the value of the DEST_DIR variable". And the leading colon on the line is a way to say process the line but don't execute it.

[cadenkriese]

Okay, I don't see the point of having that extra variable, not sure why I put it there in the first place. The docs already say just to use DEST_DIR. But thanks for the pointers anyway :)

[itzg]

Okay, I don't see the point of having that extra variable, not sure why I put it there in the first place. The docs already say just to use DEST_DIR. But thanks for the pointers anyway :)

Totally works 😃

[itzg]

Are you good with me merging or did you need to do more testing?

[itzg]

Image will get pushed with https://github.com/itzg/docker-mc-backup/actions/runs/1617547149