borgbase / ansible-role-borgbackup

Ansible role to set up Borg and Borgmatic
MIT License
226 stars 99 forks source link

Fix failing on EPEL check in Fedora #111

Closed samh closed 1 year ago

samh commented 1 year ago

The "package" install method was failing on Fedora, because it is part of the 'RedHat' OS family but does not have EPEL.

Only applies when using the "package" install method. Added a separate variable "borg_require_epel" to make it easy to disable the check in any other situations (e.g. using a custom mirror instead of the epel-release package).

m3nu commented 1 year ago

There is a big PR open right now. May integrate this PR after the bigger one to avoid conflicts.

samh commented 1 year ago

I rebased it and added the variable to the README.

m3nu commented 1 year ago

Looks good. Just added one comment to make sure the default value is clear enough?

m3nu commented 1 year ago

Thanks for the contribution!

Fixes #124