Re: svn commit: r289879 - head/usr.sbin/newsyslog

2015-10-24 Thread Jilles Tjoelker
On Sat, Oct 24, 2015 at 05:39:47PM +0200, Baptiste Daroussin wrote: > On Sat, Oct 24, 2015 at 08:31:06AM -0700, Bryan Drewery wrote: > > On 10/24/15 6:55 AM, Baptiste Daroussin wrote: > > > +static int > > > +parse_signal(const char *str) > > Maybe this and the timeout(1) version should move to li

Re: svn commit: r289879 - head/usr.sbin/newsyslog

2015-10-24 Thread Baptiste Daroussin
On Sat, Oct 24, 2015 at 08:31:06AM -0700, Bryan Drewery wrote: > On 10/24/15 6:55 AM, Baptiste Daroussin wrote: > > +static int > > +parse_signal(const char *str) > > Maybe this and the timeout(1) version should move to libc or libutil. We > do have strsignal(3) and psignal(3) too. > I was thinki

Re: svn commit: r289879 - head/usr.sbin/newsyslog

2015-10-24 Thread Bryan Drewery
On 10/24/15 6:55 AM, Baptiste Daroussin wrote: > +static int > +parse_signal(const char *str) Maybe this and the timeout(1) version should move to libc or libutil. We do have strsignal(3) and psignal(3) too. -- Regards, Bryan Drewery ___ svn-src-head@f

svn commit: r289879 - head/usr.sbin/newsyslog

2015-10-24 Thread Baptiste Daroussin
Author: bapt Date: Sat Oct 24 13:55:12 2015 New Revision: 289879 URL: https://svnweb.freebsd.org/changeset/base/289879 Log: newsyslog.conf: allow to configure the signal using the signal name. Submitted by: Alexandre Perrin MFC after:1 week Relnotes: yes Differential Revision