borgbase / ansible-role-borgbackup

Ansible role to set up Borg and Borgmatic
MIT License
229 stars 102 forks source link

Fedora: correctly install borg and borgmatic into the venv #75

Closed Ramblurr closed 3 years ago

Ramblurr commented 3 years ago

also, do not install the system packages.

fixes #74

This PR overrides #71

m3nu commented 3 years ago

We don't need python3(6)-virtualenv any more? That was another item #71 addressed.

m3nu commented 3 years ago

I'll merge this and then look into virtualenv and the errors we get after the latest Debian upgrade.

Ramblurr commented 3 years ago

We don't need python3(6)-virtualenv any more? That was another item #71 addressed.

Ah that could be the case, though I don't use RHEL directly these says.

Thanks for merging!

m3nu commented 3 years ago

I didn't get an error, so maybe it's fine to just have python-venv.