On 2016/02/01 15:02, Joerg Jung wrote:
> 
> > On 01 Feb 2016, at 14:53, Stuart Henderson <[email protected]> wrote:
> > 
> > moved from misc.
> > 
> > On 2016-01-28, Antoine Jacoutot <[email protected]> wrote:
> >>> Well, we "tradionally" had setproctitle("[priv]") in the parent.  I
> >>> changed the tradition to setproctitle("parent").
> >>> 
> >>> I have no objections with changing this in the parent (but keeping the
> >>> setproctitles in the children) to either the default (all command line
> >>> flags) or to something like setproctitle("parent, %s", conffile).
> >>> Command line flags suck and I don't think that -d or -v would be
> >>> helpful in the output, so I prefer the latter.
> >> 
> >> "-v" is helpful at least for rc.d which needs to match the full args list 
> >> by default
> >> 
> >>> All rc scripts would have to be adjusted by somebody with better rc-fu.
> >> 
> >> Actually if things are properly done, the non default pexp line in the 
> >> rc.d scripts should just be removed and that's it.
> >> 
> > 
> > Here it is for the majority of base daemons. I've left ypldap for now,
> > the rc script matches "ldap client" not "parent". radiusd already leaves
> > the parent process title as-is (the setproctitle("[main]") are separate
> > processes).
> > 
> > OK?
> 
> ok jung@
> 
> What about smtpd, should be similar, no?

Ah yes, well spotted. Missed in my grep because setproctitle isn't
called directly with the name, but in config_process() from a string
instead.

Reply via email to