Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 10 Nov 2018 08:23:46 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        Ben Woods <woodsb02@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r340318 - head/usr.sbin/newsyslog
Message-ID:  <201811101623.wAAGNkp3016454@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <201811101046.wAAAkdPs041621@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> Author: woodsb02 (ports committer)
> Date: Sat Nov 10 10:46:38 2018
> New Revision: 340318
> URL: https://svnweb.freebsd.org/changeset/base/340318
> 
> Log:
>   newsyslog.conf: Restrict included files in default config to [!.]*.conf
>   
>   The new default config will only include files from the following
>   directories which end with '.conf' and do not beginning with a '.'
>   character:
>   - /etc/newsyslog.conf.d/
>   - /usr/local/etc/newsyslog.conf.d/
>   
>   This matches the syslog.conf(5) functionality, and also prevents '.sample' or
>   '.pkgnew' files being included. This is important for ports which install files
>   in /usr/local/etc/newsyslog.conf.d/ and also for pkgbase.
>   
>   Approved by:	eadler
>   Approved by:	bapt
>   Relnotes:	yes
>   Differential Revision:	https://reviews.freebsd.org/D17086

This is going to cause people who are doing binary updates to
12.0 silent breakage in that any file not named by these rules
well suddenly and silently be ignored when they update.

My normal mode to disable files in directories of this nature
is to create a off/ directory and mv the files to there rather
than rename the file in place and depend on special rules like
this to deal with it.

If we need a palce for examples I would support a examples/
subdirectory for them, which would de-clutter the working
directory in actual use at the same time.


> 
> Modified:
>   head/usr.sbin/newsyslog/newsyslog.8
>   head/usr.sbin/newsyslog/newsyslog.conf
> 
> Modified: head/usr.sbin/newsyslog/newsyslog.8
> ==============================================================================
> --- head/usr.sbin/newsyslog/newsyslog.8	Sat Nov 10 10:32:55 2018	(r340317)
> +++ head/usr.sbin/newsyslog/newsyslog.8	Sat Nov 10 10:46:38 2018	(r340318)
> @@ -17,7 +17,7 @@
>  .\" the suitability of this software for any purpose.  It is
>  .\" provided "as is" without express or implied warranty.
>  .\"
> -.Dd July 19, 2018
> +.Dd November 10, 2018
>  .Dt NEWSYSLOG 8
>  .Os
>  .Sh NAME
> @@ -268,10 +268,12 @@ will examine all files listed in the configuration fil
>  .Nm
>  configuration file
>  .It Pa /etc/newsyslog.conf.d
> -Each file in this directory will be included by the default
> +By default each file in this directory ending in '.conf' and not beginning with
> +a '.' will be included by the default
>  .Pa newsyslog.conf .
>  .It Pa /usr/local/etc/newsyslog.conf.d
> -Each file in this directory will be included by the default
> +By default each file in this directory ending in '.conf' and not beginning with
> +a '.' will be included by the default
>  .Pa newsyslog.conf .
>  .El
>  .Sh COMPATIBILITY
> 
> Modified: head/usr.sbin/newsyslog/newsyslog.conf
> ==============================================================================
> --- head/usr.sbin/newsyslog/newsyslog.conf	Sat Nov 10 10:32:55 2018	(r340317)
> +++ head/usr.sbin/newsyslog/newsyslog.conf	Sat Nov 10 10:46:38 2018	(r340318)
> @@ -31,5 +31,5 @@
>  /var/log/utx.log			644  3	   *	@01T05 B
>  /var/log/weekly.log			640  5	   *	$W6D0 JN
>  
> -<include> /etc/newsyslog.conf.d/*
> -<include> /usr/local/etc/newsyslog.conf.d/*
> +<include> /etc/newsyslog.conf.d/[!.]*.conf
> +<include> /usr/local/etc/newsyslog.conf.d/[!.]*.conf
> 
> 

-- 
Rod Grimes                                                 rgrimes@freebsd.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201811101623.wAAGNkp3016454>