Date: Wed, 5 Feb 2020 14:35:03 -0500 From: Mathieu <sigsys@gmail.com> To: Andrew Gierth <andrew@tao11.riddles.org.uk>, Kyle Evans <kevans@freebsd.org> Cc: Ian Lepore <ian@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r357566 - head Message-ID: <578cece7-90e4-afe3-b6e5-3b0961081942@gmail.com> In-Reply-To: <874kw4ooqw.fsf@news-spur.riddles.org.uk> References: <202002050443.0154hwPl091233@repo.freebsd.org> <1c19eca5b40aa7b70b0c18b7b52b540671cc50dd.camel@freebsd.org> <CACNAnaHu=T9N5S6DKdYN9FjRBb%2BASvEU7XPVC6x3jDZdkjtw1g@mail.gmail.com> <874kw4ooqw.fsf@news-spur.riddles.org.uk>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2/5/20 2:05 PM, Andrew Gierth wrote: >>>>>> "Kyle" == Kyle Evans <kevans@freebsd.org> writes: > >> Unfortunately, the utility of this cool series of changes is mostly > >> destroyed by the fact that PATH is the variable that could be most > >> usefully set in login.conf for running daemons, specifically because > >> it is set in 3 separate places now. > > Kyle> It is (and has been) set in each of them respectively, but the > Kyle> submitter (along with sigsys@gmail.com) have some idea to accept > Kyle> a hint in cron (and perhaps the others) from login.conf that it > Kyle> should accept PATH as well unless it's been explicitly set in the > Kyle> executing job. > > Kyle> I've CC'd these two so they can expand on that or discuss as > Kyle> needed. > > I'd be quite happy to go along with a proposal to unconditionally set > PATH from login.conf in cron and elsewhere. > > (The idea of having a flag for it is just to avoid changing existing PATH > behavior without an explicit admin choice, to head off arguments about > security. If that's not an issue, then no need for the flag.) > If there are objections to using the "daemon" class's PATH for rc/service, there could be a new "system" class added that inherits from "daemon" configured with the traditional PATH that they override with (i.e. without /usr/local paths). That would require no other changes to rc/service after modifying them to not always override PATH. And leave only the cron issue.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?578cece7-90e4-afe3-b6e5-3b0961081942>