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

Warner Losh imp at bsdimp.com
Sat May 13 16:39:17 UTC 2017


On Sat, May 13, 2017 at 10:32 AM, Ngie Cooper <yaneurabeya at gmail.com> wrote:
>
>> On May 13, 2017, at 09:29, Ngie Cooper <yaneurabeya at gmail.com> wrote:
>>
>> On May 13, 2017, at 08:37, Rodney W. Grimes <freebsd at pdx.rh.CN85.dnsmgr.net> 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 consumers of FreeBSD to use the unmodified configuration
>>>> files out of the box more than previously.
>>>
>>> What about simply generating proper newsyslog.conf and syslog.conf based
>>> on the the MK_ values rather than change the visible administration
>>> interface that has finger memory, ansible, and puppet support?
>>
>> - My employer doesn't use puppet on its appliances.
>> - Files snapshotted in time bitrot (which was the case with our configs, which is part of the reason behind this change.

couldn't you have an Isilion config file in the appropriate directory
instead? And use the default newsyslog / syslog config files?

>> - 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 etc/Makefile, which is already complicated enough as-is.

Why do you care about removing them at all? They are no-ops if the
files don't exist. Why not just always install all these files is
where I'm going with this...

Warner


More information about the svn-src-all mailing list