fix(sysrc-svc): workaround *BSD minion indefinitely hanging on start

[sticky-note]

Check saltstack/salt#44848 for further details

[alxwr]

@vutny If you don't have any FreeBSD hosts to test this PR: I've got them and I use this formula. :-)
@sticky-note Thanks for this!

[sticky-note]

@alxwr Is this the correct way to do this ? I've seen you changed the way to do this on https://github.com/saltstack-formulas/prometheus-formula but don't know if it is applicable here

[alxwr]

@sticky-note I still use >/dev/null 2>&1. Just refactored my code a bit.

[alxwr]

@sticky-note Just tested this PR on FreeBSD 11.2. Works as expected. Thanks for this!
Please just make the service watch the sysrc flags.

Otherwise: LGTM

[sticky-note]

@alxwr Updated

[myii]

@sticky-note We've got semantic-release active on this repo (it looks like you already know this). The commit message is just failing the commitlint because of the capital letter. If you can amend the commit and then force push it back again, it will ensure you get a release once this PR is merged:

-fix(sysrc-svc): Workaround *BSD minion undefinitely hanging on start
+fix(sysrc-svc): workaround *BSD minion indefinitely hanging on start

---

Update: Just a little typo fix in the commit message as well.

[vutny]

Fine for me, since there are no regressions. Thanks @sticky-note .
Just need to fix that minor issue with commit message for all tests to pass.

[myii]

@alxwr @vutny Thanks for the reviews.
@sticky-note Thanks for the contribution -- merged.

[saltstack-formulas-travis]

🎉 This PR is included in version 0.37.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀