borgmatic-collective / docker-borgmatic

Borgmatic in Docker

GNU General Public License v3.0

313 stars 88 forks source link

change CRON ENV to BACKUP_CRON as stated in README.md #328

Closed chrisblech closed 1 week ago

chrisblech commented 1 month ago

As of README.md, cron time can be set via ENV BACKUP_CRON, but init script uses CRON instead.

This commit changes init script, so BACKUP_CRON is used.

Additional minor changes:

docke-compose.yml: remove volume for /root/.borgmatic (as README.md says this is redundant and should be removed)
fix README.md: script lives in /usr/local/bin/ instead of /usr/bin/
fix README.md: in default crontab.txt, there is no $PATH set

Psycho0verload commented 3 weeks ago

I plan to open another comprehensive and corresponding pull request for you tonight, probably at night. It will also contain the changes from this one and expand it a little.

331

Psycho0verload commented 1 week ago

@chrisblech Your changes are now included in the master. I have included it in #331. So you can close this PR