Date: Thu, 12 Jul 2012 13:45:13 -0700 From: Doug Barton <dougb@dougbarton.us> To: Michael Scheidell <scheidell@FreeBSD.org> Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/security/snortsam/files snortsam.sh.in Message-ID: <4FFF3759.80806@dougbarton.us> In-Reply-To: <201207121951.q6CJphl4049055@repoman.freebsd.org> References: <201207121951.q6CJphl4049055@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
This is a bad change, and probably does not do what you're intending. The proper way to handle this would be to use REQUIRE: LOGIN. It's also worth noting that the reasons for using REQUIRE: LOGIN by default for ports scripts unless there is a good reason not to include grouping them together rather than scattering them throughout the boot process. It's not just a matter of unprivileged users. Please see http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/rc-scripts.html for more information. Doug On 07/12/2012 12:51 PM, Michael Scheidell wrote: > scheidell 2012-07-12 19:51:43 UTC > > FreeBSD ports repository > > Modified files: > security/snortsam/files snortsam.sh.in > Log: > - remove rcscript REQUIRE: LOGIN > > PR: ports/169524 http://www.FreeBSD.org/cgi/query-pr.cgi?pr=169524 > Approved by: maintainer (timeout, 14 days) > > Revision Changes Path > 1.6 +1 -2 ports/security/snortsam/files/snortsam.sh.in > > http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/security/snortsam/files/snortsam.sh.in.diff?&r1=1.5&r2=1.6&f=u >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4FFF3759.80806>