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

[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 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...

I think this is actually the bigger question.  I think it is perfectly
sensible to support conf.d/* files for ports to use and as a way to
manage logs for application logs on an appliance, etc.  However, this
shuffling is a bit of a merge nightmare for anyone using mergemaster
or etcupdate, and the biggest cost is that newsyslog will create a
one-line file in /var/log for entries with 'C'.

-- 
John Baldwin