Date: Mon, 9 May 2011 20:21:45 -0400 From: Jason Hellenthal <jhell@DataIX.net> To: Gordon Tetlow <gordon@tetlows.org> Cc: freebsd-rc@freebsd.org Subject: Re: [RFC][Change-Request] Create usefulness in rc.subr etc/rc.conf.d/*.conf namespace. Message-ID: <20110510002145.GD2558@DataIX.net> In-Reply-To: <BANLkTin679hJDiAB8bZkaUER2L7bC97x5Q@mail.gmail.com> References: <20110508191336.GC3527@DataIX.net> <BANLkTi=hozQBLUC15NsF2rky2OfFW=t_RQ@mail.gmail.com> <01d201cc0e6c$47d4b180$d77e1480$@vicor.com> <BANLkTimqhu215ZdHwx=fHqu33NXj2pYpvw@mail.gmail.com> <20110509190441.GC82456@DataIX.net> <BANLkTin679hJDiAB8bZkaUER2L7bC97x5Q@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --]
Gordon,
On Mon, May 09, 2011 at 04:50:57PM -0700, Gordon Tetlow wrote:
> On Mon, May 9, 2011 at 12:04 PM, Jason Hellenthal <jhell@dataix.net> wrote:
> >
> > Gordon,
> >
> > On Mon, May 09, 2011 at 10:32:18AM -0700, Gordon Tetlow wrote:
> >> On Mon, May 9, 2011 at 10:12 AM, Devin Teske <dteske@vicor.com> wrote:
> >> > The solution is to have a script that can tell you these two details:
> >> >
> >> > 1. What is the final value of ``*_enable''
> >> > 2. Which file assigns said final value
> >> >
> >> > If you have those two pieces of information, then unraveling a twisted
> >> > configuration is easier.
> >> >
> >> > [Re-]welcome my sysrc(8) script:
> >>
> >> While this is a very cool script, I have to wonder how far we have
> >> strayed if we require another tool to tell us how the system is
> >> configured. Surely we should be aiming for simplicity in our
> >> implementation and not something incredibly complex.
> >>
> >> After Jason's proposal, we would have the following list of configuration files:
> >>
> >> /etc/defaults/rc.conf
> >> /etc/rc.conf
> >> /etc/rc.conf.local
> >
> > What seems to be lost here is that the below two are "optional" not
> > something that should be created by anything other than the user who needs
> > that functionality. Yes having two of the below is a problem. Yes {name}
> > needs to go. But until there is something to replace it in a way that is
> > agreed on we cant get rid of the broken {name} implement.
>
> The {name} implementation isn't broken, it just doesn't do what you want it to.
>
> I would be hesitant to remove the {name} implementation, it's been
> there since the 5.x days. It's hard to say how many installations rely
> on it being there.
>
Though I would like to I agree with that. You dint just introduce an
interface and remove it later down the road with a long winded deprecation
process. This certainly could be the start of that deprecation process.
"It just doesn't do what it is suggested to do" not what I want it to do
by rc.conf(5) which is part of the reason why this patch came into play.
You can't just provide a space to use and then suggest to the user that
they only use certain undocumented names that are provided by rc.d/*
scripts. It's a waste of their time thinking that they can just put for
instance 'jail, jail1, jail2' in rc.conf.d and it'll work. It does not
unless you start adding source this and that lines to the end of every
file and that would not be right to suggest to the end-user either.
Though if it is documented well enough I do not see any fathomable reason
why a user should not be allowed to make up a name that contains what they
want to be in it. The rc.conf.d directory suggests its very use just as
anyone would believe about usr/local/etc/apache22/Includes/*.conf.
Out of all the opposition yet I still have not seen a valid response of
"what is wrong with letting the user decide how they name a configuration
file?". I keep seeing the opposite of well you can do this and you can do
that with shell scripting options. That shocks me as this is like stated
before a user specified created and maintained directory.
--
Regards, (jhell)
Jason Hellenthal
[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (FreeBSD)
Comment: http://bit.ly/0x89D8547E
iQEcBAEBAgAGBQJNyIUYAAoJEJBXh4mJ2FR+t/sH/igSxUPV66powmaOT8XRtGa9
ZbOmz+eQwy5vr2+eGs3bhs4GmXdcKErcHG5Jvj7cLtI5F28VVtimKE9yiNO+3bKM
5I1LA6uF7TiU3tpZ8WuKgNZBCam3YU/DC4lFKdvwnmLYGIJzYWRU7ofljj7jB1ck
mOUzXUyOndEJ88hUyDTsroay+mnap8RwSPqVTBWoxakNMCdMTPI1TyR1EKv75QtF
7nb9SmH/veEeFMJ5MiDT5qWJUQLPNObm1/5v5cpnOSOjfH4k/9e20PjD2U5jSku1
pmDCUoWbG8gsVzkIKB+k+wl4TuinZXYYMSp7YheP7V466IOSdUj38yLSqT14TUo=
=0dti
-----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110510002145.GD2558>