New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pure-ftpd: fix launchd plist #20124
pure-ftpd: fix launchd plist #20124
Conversation
Correct or document several install issues with pure-ftpd - plist requires each argument be added as an individual element [1] - add default users database location to plist configration [2] - document that service must be started as root [3] - `-` arguments replaced in plist with equivalent `--` arguments for better readability [1] https://apple.stackexchange.com/questions/284209/having-trouble-configuring-homebrew-to-run-pure-ftpd-as-a-service [2] https://github.com/jedisct1/pure-ftpd/issues/39 [3] https://github.com/jedisct1/pure-ftpd/issues/38
fix launchd plist
Formula/pure-ftpd.rb
Outdated
@@ -40,6 +40,16 @@ def install | |||
system "make", "install" | |||
end | |||
|
|||
def caveats |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We prefer not to add new caveats to formulae, since they become outdated quickly and add to maintenance overhead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed caveat block.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revised PR per your request.
Thanks! |
Thanks for your first contribution to Homebrew, @uriahcarpenter! |
Correct or document several install issues with pure-ftpd
-
arguments in plist with equivalent long--
arguments for better readability [4][1] https://apple.stackexchange.com/questions/284209/having-trouble-configuring-homebrew-to-run-pure-ftpd-as-a-service
[2] https://github.com/jedisct1/pure-ftpd/issues/39
[3] https://github.com/jedisct1/pure-ftpd/issues/38
[4] https://download.pureftpd.org/pub/pure-ftpd/doc/README
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install <formula>
)?