Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 04 Jun 2017 16:05:08 -0700
From:      John Baldwin <jhb@freebsd.org>
To:        "Ngie Cooper (yaneurabeya)" <yaneurabeya@gmail.com>
Cc:        Baptiste Daroussin <bapt@freebsd.org>, rgrimes@freebsd.org, Ngie Cooper <ngie@freebsd.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r318441 - in head/etc: . cron.d
Message-ID:  <4569582.oWoS6TS87T@ralph.baldwin.cx>
In-Reply-To: <16687E47-B9A1-4386-BFCD-FB9E1F5C3D90@gmail.com>
References:  <201705180625.v4I6Pd9j062495@repo.freebsd.org> <1833775.llHFF7vaD6@ralph.baldwin.cx> <16687E47-B9A1-4386-BFCD-FB9E1F5C3D90@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Saturday, June 03, 2017 01:49:34 PM Ngie Cooper wrote:
>=20
> > On Jun 3, 2017, at 12:05, John Baldwin <jhb@freebsd.org> wrote:
>=20
> =E2=80=A6
>=20
> > Ping?  bapt@ agreed with this, but I haven't seen any other feedbac=
k?  If others
> > agree then we should either revert back to a single foo.conf or we =
should fully
> > split newsyslog.conf and syslogd.conf out to individual files.
>=20
> Please, no. The single monolithic foo.conf idea may have made sense b=
ack in the day, but with the various knobs that are available for build=
ing the system, for ease-of-use/intuitiveness the feature makes sense.
> I planned on introducing more of these files for other areas of the s=
ystem that aren=E2=80=99t currently being handled=E2=80=94 like bsnmpd=E2=
=80=99s logging, etc. I think the modularization makes sense.

To be clear, I listed two options, and you have only implemented a thir=
d
(which is to mix the two styles).  I think mixing the two styles is the=

absolute worst thing to do.  If we are going to split a given foo.conf
up we should split it up fully so that the out-of-box configuration for=

a given daemon is consistent.

--=20
John Baldwin



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