NewNewYorkBridges / ansible-tor-bridge

Ansible role to deploy Tor Bridge nodes.
GNU General Public License v3.0
7 stars 5 forks source link

Add support for FreeBSD #2

Closed pjsier closed 2 years ago

pjsier commented 2 years ago

Related to #1, adds support for FreeBSD. One of the main changes I made is splitting out OS-specific variables files and loading them with include_vars, and I based that on nusenu/ansible-relayor. I tested this on FreeBSD 12.2 and 13.0 which is why I included those versions in meta.yml.

While going through the FreeBSD instructions it mentioned including the line "Log notice file" in torrc. Do you think it would make sense to update that here or make it configurable?

I'm happy to make any changes, thanks!

pjsier commented 2 years ago

@lilianbaz bumping this, let me know if there's anything I should change, thanks!

lilianbaz commented 2 years ago

Hi @pjsier Thank you very much for your help, and sorry for the delay. I just wanted to test it before merge, but I didn't have FreeBSD nodes at the moment!

pjsier commented 2 years ago

@lilianbaz no problem at all, thanks for the review! I'm interested in adding support for more operating systems if that's of interest