Date: Thu, 22 Apr 2010 09:26:23 -0400 From: John Baldwin <jhb@freebsd.org> To: freebsd-current@freebsd.org Cc: Gordon Tetlow <gordon@freebsd.org> Subject: Re: newsyslog patch implementing file includes Message-ID: <201004220926.23593.jhb@freebsd.org> In-Reply-To: <y2i4e571dd71004212055v17af3f28v172b8f8204b0b7e8@mail.gmail.com> References: <y2i4e571dd71004212055v17af3f28v172b8f8204b0b7e8@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday 21 April 2010 11:55:44 pm Gordon Tetlow wrote: > I wanted the ability for a port to have a rotating log policy so I wrote a > patch for newsyslog to implement includes of other newsyslog.conf style > files. > > Please find the patch at: > http://people.freebsd.org/~gordon/patches/newsyslog.diff<http://people.freebsd.org/%7Egordon/patches/newsyslog.diff>; > > Format for the include line in /etc/newsyslog.conf is: > <include> /etc/defaults/newsyslog.conf > > Here's a quick overview of the changes: > Convert the conf_entry struct from using a home rolled linked list to the > queue(3) macros. > Add a STAILQ to process include files. > Add support for <include> tag to specify include files. > Globbing is supported in <include> statements. > Properly detect circular include loop dependencies. > > Please take a look and send me any comments you might have. This is a great feature! One suggestion, I think this text in the new manpage isn't quite right: Name of the system log file to be archived, the literal string "default", or "include". I think it's ambiguous about "include" also being a literal string. Two possible suggestions: Name of the system log file to be archived, or one of the literal strings "default" or "include". Name of the system log file to be archived, the literal string "default", or the literal string "include". -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201004220926.23593.jhb>