From owner-svn-src-stable-10@freebsd.org Thu Jun 22 07:35:24 2017 Return-Path: Delivered-To: svn-src-stable-10@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id B9973DA83D7; Thu, 22 Jun 2017 07:35:24 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from mail.baldwin.cx (bigwig.baldwin.cx [96.47.65.170]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 757AC7110A; Thu, 22 Jun 2017 07:35:23 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from John-Baldwins-MacBook-Pro-2.local (d-24-245-127-120.cpe.metrocast.net [24.245.127.120]) by mail.baldwin.cx (Postfix) with ESMTPSA id 1ADAB10AB01; Thu, 22 Jun 2017 03:35:16 -0400 (EDT) Subject: Re: svn commit: r320222 - in stable/10: etc etc/cron.d etc/mtree etc/newsyslog.conf.d etc/pam.d etc/syslog.d tools/build/mk usr.sbin/cron/cron usr.sbin/cron/lib usr.sbin/syslogd To: Ngie Cooper , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org References: <201706220708.v5M78IYv080609@repo.freebsd.org> From: John Baldwin Message-ID: Date: Thu, 22 Jun 2017 03:35:15 -0400 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <201706220708.v5M78IYv080609@repo.freebsd.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.4.3 (mail.baldwin.cx); Thu, 22 Jun 2017 03:35:16 -0400 (EDT) X-Virus-Scanned: clamav-milter 0.99.2 at mail.baldwin.cx X-Virus-Status: Clean X-BeenThere: svn-src-stable-10@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for only the 10-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 Jun 2017 07:35:24 -0000 On 6/22/17 3:08 AM, Ngie Cooper wrote: > Author: ngie > Date: Thu Jun 22 07:08:18 2017 > New Revision: 320222 > URL: https://svnweb.freebsd.org/changeset/base/320222 > > Log: > MFC r308139,r308157,r308160,r316818,r318250,r318443: > > r308139 (by bapt): > > cron(8): add support for /etc/cron.d and /usr/local/etc/cron.d > > For automation tools it is way easier to maintain files in directories rather > than modifying /etc/crontab. > > The files in those directories are in the same format as /etc/crontab > > Relnotes: yes > > r308157 (by bapt): > > Fix typo in cron(8) date > > r308160 (by bapt): > > syslogd(8): add an 'include' keyword > > All the '.conf' files not beginning with a '.' contained int he directory > following the keyword will be included. > > This keyword can only be used in the first level configuration files. > > Modify the default syslogd.conf to 'include' /etc/syslog.d and > /usr/local/etc/syslog.d > > It simplify a lot handling of syslog from automation tools. > > Relnotes: yes > > r316818: > > Conditionally install /etc/pam.d/ftp* and /etc/pam.d/telnetd > > /etc/pam.d/ftp* should be installed with MK_FTP != no and > /etc/pam.d/telnetd should be installed when MK_TELNET != no. > > r318250: > > 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. > > Both newsyslog.conf and syslog.conf: > - /var/log/lpd-errs (MK_LPR != no) > - /var/log/ppp.log (MK_PPP != no) > - /var/log/xferlog (MK_FTP != no) > > newsyslog.conf: > - /var/log/amd.log (MK_AMD != no) > - /var/log/pflog (MK_PF != no) > - /var/log/sendmail.st (MK_SENDMAIL != no) > > r318443: > > Conditionally handle the crontab entry for atrun(8) > > The default crontab prior to this commit assumes atrun(8) is always > present, which isn't true if MK_AT == no. Move atrun(8) execution > from /etc/crontab to /etc/cron.d/at, and base /etc/cron.d/at's installation > on MK_AT. cron(8) will detect /etc/cron.d/at's presence when the configuration > is loaded and run atrun every 5 minutes like it would prior to this commit. > > SHELL and PATH are duplicated between /etc/crontab and /etc/cron.d/at > because atrun(8) executes programs, which may rely on environment > set in the current default /etc/crontab. > > Noted by: bdrewery (in an internal review) > Relnotes: yes (may need to add environmental modifications to > /etc/cron.d/at) Please revert the breakout of the existing config files. I think that splitting up the conf files is too disruptive of a change (POLA) for stable branches. This was agreed to by other folks in the followup thread to the commits in head that you continue to ignore. In head please either fully split up the files or revert to a single file (another part of that thread you continue to ignore). -- John Baldwin