Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-19 Thread Julian Elischer
On 16/5/17 4:11 am, Ian Lepore wrote: On Mon, 2017-05-15 at 12:22 -0700, John Baldwin wrote: On Monday, May 15, 2017 11:43:21 AM Ian Lepore wrote: On Mon, 2017-05-15 at 10:13 -0700, John Baldwin wrote: On Saturday, May 13, 2017 10:39:15 AM Warner Losh wrote: - It's really easy to screw

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ian Lepore
On Mon, 2017-05-15 at 21:09 +, Alexey Dokuchaev wrote: > On Mon, May 15, 2017 at 02:49:30PM -0600, Ian Lepore wrote: > > > > ... > > You acknowledge that the situation is different for ports, so does that > > mean your objections go away when base becomes packaged and faces the > > same

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Alexey Dokuchaev
On Mon, May 15, 2017 at 08:28:53PM +, Alexey Dokuchaev wrote: > On Mon, May 15, 2017 at 02:11:53PM -0600, Ian Lepore wrote: > > ... > > Sure, all us old-timers have the finger memory for editing rc.conf and > > syslog.conf and so on, but how often do you crack open syslog.conf with > > the

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread John Baldwin
On Monday, May 15, 2017 02:11:53 PM Ian Lepore wrote: > On Mon, 2017-05-15 at 12:22 -0700, John Baldwin wrote: > > Also, _you_ could just splat an empty /etc/newsyslog.conf file on > > your > > appliance and create a bunch of conf.d/foo files if that is easier > > for > > you to use on an

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Alexey Dokuchaev
On Mon, May 15, 2017 at 02:49:30PM -0600, Ian Lepore wrote: > ... > You acknowledge that the situation is different for ports, so does that > mean your objections go away when base becomes packaged and faces the > same installation and update issues that packaged ports do? Because I > was under

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ian Lepore
On Mon, 2017-05-15 at 20:28 +, Alexey Dokuchaev wrote: > On Mon, May 15, 2017 at 02:11:53PM -0600, Ian Lepore wrote: > > > > ... > > In general a lot of this feels like "I only needed 6 big config > > files > > to control my whole system in 1988, and so I should only need those > > same 6

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Alexey Dokuchaev
On Mon, May 15, 2017 at 02:11:53PM -0600, Ian Lepore wrote: > ... > In general a lot of this feels like "I only needed 6 big config files > to control my whole system in 1988, and so I should only need those > same 6 files now." Yup; doesn't it feel good? (That's one of our selling points BTW.)

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ian Lepore
On Mon, 2017-05-15 at 14:11 -0600, Ian Lepore wrote: > On Mon, 2017-05-15 at 12:22 -0700, John Baldwin wrote: > > > > On Monday, May 15, 2017 11:43:21 AM Ian Lepore wrote: > > > > > > [...] > > > > Also, _you_ could just splat an empty /etc/newsyslog.conf file on > > your > > appliance and

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ian Lepore
On Mon, 2017-05-15 at 12:22 -0700, John Baldwin wrote: > On Monday, May 15, 2017 11:43:21 AM Ian Lepore wrote: > > > > On Mon, 2017-05-15 at 10:13 -0700, John Baldwin wrote: > > > > > > On Saturday, May 13, 2017 10:39:15 AM Warner Losh wrote: > > > > > > > > > > > > > > > > > > > > > > > >

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread John Baldwin
On Monday, May 15, 2017 11:43:21 AM Ian Lepore wrote: > On Mon, 2017-05-15 at 10:13 -0700, John Baldwin wrote: > > On Saturday, May 13, 2017 10:39:15 AM Warner Losh wrote: > > > > > > > > > > > > > > > > > - It's really easy to screw up a mergemaster call if you edit > > > > > the files, and

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ian Lepore
On Mon, 2017-05-15 at 11:24 -0700, Ngie Cooper (yaneurabeya) wrote: > > > > On May 15, 2017, at 10:47, Ngie Cooper (yaneurabeya) > mail.com> wrote: > > > > > > > > > > On May 15, 2017, at 10:43, Ian Lepore wrote: > > … > > > > > > > > That's only a good

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Bryan Drewery
On 5/12/2017 8:10 PM, Ngie Cooper wrote: > Author: ngie > Date: Sat May 13 03:10:50 2017 > New Revision: 318250 > URL: https://svnweb.freebsd.org/changeset/base/318250 > > Log: > Handle the logfiles in newsyslog and syslogd conditionally, based on > src.conf(5) knobs > > This will allow

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ngie Cooper (yaneurabeya)
> On May 15, 2017, at 10:47, Ngie Cooper (yaneurabeya) > wrote: > > >> On May 15, 2017, at 10:43, Ian Lepore wrote: > > … > >> That's only a good argument for keeping the lines in the monolithic >> file if those lines will be ignored when a file in

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ngie Cooper (yaneurabeya)
> On May 15, 2017, at 10:43, Ian Lepore wrote: … > That's only a good argument for keeping the lines in the monolithic > file if those lines will be ignored when a file in the .conf.d > directory provides conflicting config. Otherwise my embedded product > that drops

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread Ian Lepore
On Mon, 2017-05-15 at 10:13 -0700, John Baldwin wrote: > On Saturday, May 13, 2017 10:39:15 AM Warner Losh wrote: > > > > > > > > > > > > > - It's really easy to screw up a mergemaster call if you edit > > > > the files, and install the stock version which removes the > > > > edits. > > > Also,

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-15 Thread John Baldwin
On Saturday, May 13, 2017 10:39:15 AM Warner Losh wrote: > >> - It's really easy to screw up a mergemaster call if you edit the files, > >> and install the stock version which removes the edits. > > > > Also, programmatically removing the entries means you have to bake the > > metadata into

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Nikolai Lifanov
On 05/13/2017 13:21, Ngie Cooper (yaneurabeya) wrote: > Even ansible/chef/puppet would have to bake the configuration removal logic > into its template files, which seems like a pain for folks (and the same > logic would need to be implemented multiple times instead of once). Having to template

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Ngie Cooper (yaneurabeya)
> On May 13, 2017, at 9:51 AM, Alexey Dokuchaev wrote: > > On Sat, May 13, 2017 at 10:24:20AM -0600, Ian Lepore wrote: >> ... >> The evolution for years has been away from monolithic config files >> containing a mashup of values for unrelated subsystems and towards >> .conf.d

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Ngie Cooper (yaneurabeya)
> On May 13, 2017, at 9:39 AM, Warner Losh wrote: > > On Sat, May 13, 2017 at 10:32 AM, Ngie Cooper > wrote: >> >>> On May 13, 2017, at 09:29, Ngie Cooper wrote: >>> >>> On May 13, 2017, at 08:37,

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Alexey Dokuchaev
On Sat, May 13, 2017 at 10:24:20AM -0600, Ian Lepore wrote: > ... > The evolution for years has been away from monolithic config files > containing a mashup of values for unrelated subsystems and towards > .conf.d directories containing many single-subject files. This "evolution" had probably

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Warner Losh
On Sat, May 13, 2017 at 10:32 AM, Ngie Cooper wrote: > >> On May 13, 2017, at 09:29, Ngie Cooper wrote: >> >> On May 13, 2017, at 08:37, Rodney W. Grimes >> wrote: >> Author: ngie Date: Sat May 13 03:10:50

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Ngie Cooper
> On May 13, 2017, at 09:29, Ngie Cooper wrote: > > On May 13, 2017, at 08:37, Rodney W. Grimes > wrote: > >>> Author: ngie >>> Date: Sat May 13 03:10:50 2017 >>> New Revision: 318250 >>> URL:

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Ngie Cooper
On May 13, 2017, at 08:37, Rodney W. Grimes wrote: >> Author: ngie >> Date: Sat May 13 03:10:50 2017 >> New Revision: 318250 >> URL: https://svnweb.freebsd.org/changeset/base/318250 >> >> Log: >> Handle the logfiles in newsyslog and syslogd conditionally, based

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Alexey Dokuchaev
On Sat, May 13, 2017 at 08:37:42AM -0700, Rodney W. Grimes wrote: > > New Revision: 318250 > > URL: https://svnweb.freebsd.org/changeset/base/318250 > > > > Log: > > Handle the logfiles in newsyslog and syslogd conditionally, based on > > src.conf(5) knobs > > > > This will allow

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Ian Lepore
On Sat, 2017-05-13 at 08:37 -0700, Rodney W. Grimes wrote: > > > > Author: ngie > > Date: Sat May 13 03:10:50 2017 > > New Revision: 318250 > > URL: https://svnweb.freebsd.org/changeset/base/318250 > > > > Log: > >   Handle the logfiles in newsyslog and syslogd conditionally, based on > >  

Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-13 Thread Rodney W. Grimes
> Author: ngie > Date: Sat May 13 03:10:50 2017 > New Revision: 318250 > URL: https://svnweb.freebsd.org/changeset/base/318250 > > Log: > Handle the logfiles in newsyslog and syslogd conditionally, based on > src.conf(5) knobs > > This will allow consumers of FreeBSD to use the

svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk

2017-05-12 Thread Ngie Cooper
Author: ngie Date: Sat May 13 03:10:50 2017 New Revision: 318250 URL: https://svnweb.freebsd.org/changeset/base/318250 Log: Handle the logfiles in newsyslog and syslogd conditionally, based on src.conf(5) knobs This will allow consumers of FreeBSD to use the unmodified configuration