Subject: Re: CVS commit: basesrc/etc/rc.conf.d
To: Darren Reed <darrenr@reed.wattle.id.au>
From: Luke Mewburn <lukem@wasabisystems.com>
List: source-changes
Date: 04/18/2002 20:33:55
On Thu, Apr 18, 2002 at 07:44:15PM +1000, Darren Reed wrote:
  | I'm going to back it out but not for any reason any of you have cited
  | and not least because anyone on icb thought it would be the right thing
  | to do when I suggested it (they just continued to whinge) but out of
  | respect that 1.6 should be not very far away and this sort of change
  | is really quite stupid in light of an imminent release.  Although it
  | did seem like a good idea at the time but so did blowing up Israel.

It's already been backed out.

For what it's worth, inflammatory comments about worldwide political
events have no place on NetBSD discussion forums.  It's like bringing
in the proverbial nazism analogies.


  | Luke was going to send me his thoughts on this but he's neglected to do
  | so.

By the time I read my email today, other people's email on the topic
had already covered what I was going to say, which was that your
change was inappropriate and should have been discussed first. 
I had already mentioned some of those thoughts to to you yesterday in
a quick chat after the commit.


  | I'll have to remember to disregard what he says in future about
  | what NetBSD supports (you can blame him for putting the seed of this
  | in my mind by mentioning NetBSD supported this earlier in the week,
  | which I took as an indication that it was ok for NetBSD to use.  Stupid
  | me for trusting heresay and a few lines of shell code.)

Your blameshifting is inappropriate.  It's your fault for:
	+ not doing appropriate research into the prior art of what
	  NetBSD was currently doing
	+ not discussing such a change when you've admitted that you
	  didn't fully understand what was going on (by giving
	  spurious reasons like "you didn't have time to ask" or
	  whatever).

Your change was not that drastically important that the time to *ask*
for clarification about various implementation issues would have
cost you anything.

The documentation for rc.conf.d isn't as complete as it could be
(I'm working on that), but it is covered in rc.subr(8) under
"load_rc_config"...

Luke.