- Add trivial rc script - Fix portlint
Created attachment 210735 [details] port patch
I don't think having it run as root by default is anything like a good idea.
Created attachment 210809 [details] new port patch ok, added separate user and group
user and group ids are finite resources, I am not sure eating one user and one group out of the limited pool we have is a great idea, it would probably be better to require a non root user, but let the user use an existing one or create one.
(In reply to Mathieu Arnold from comment #4) Well, you have just suggested what I had in the first patch. btpd_user was set to root in rc script by default. But the user can redefine it. Am I missing the idea?
(In reply to Mathieu Arnold from comment #4) Do you mean to set btpd_user to something like "changemeplease" in rc script by default? This would leave it non-working state out of the box. However we could have some user existence check in rc script. Isn't it too much for rc script?
A commit references this bug: Author: mat Date: Tue Feb 11 14:16:40 UTC 2020 New revision: 525825 URL: https://svnweb.freebsd.org/changeset/ports/525825 Log: Add a rc startup script. Leave the user creation to the end-user. PR: 243352 Submitted by: timp87 gmail com Changes: head/net-p2p/btpd/Makefile head/net-p2p/btpd/files/ head/net-p2p/btpd/files/btpd.in
Hi, mat@! I just realized that this https://svnweb.freebsd.org/ports/head/net-p2p/btpd/files/btpd.in?view=markup&pathrev=525825#l34 should be '-z' instead of '-n'