svn commit: r357566 - head

[Mathieu]

On 2/5/20 2:05 PM, Andrew Gierth wrote:
>>>>>> "Kyle" == Kyle Evans <kevans at 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 at 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.