Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 11 Jan 2001 22:36:17 +0200
From:      Sheldon Hearn <sheldonh@uunet.co.za>
To:        Matt Dillon <dillon@earth.backplane.com>
Cc:        obrien@freebsd.org, Doug Barton <dougb@freebsd.org>, cvs-committers@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/etc crontab rc src/etc/defaults rc.conf src/etc/mtree BSD.root.dist src/libexec Makefile src/libexec/save-entropy Makefile save-entropy.sh 
Message-ID:  <10047.979245377@axl.fw.uunet.co.za>
In-Reply-To: Your message of "Thu, 11 Jan 2001 12:09:14 PST." <200101112009.f0BK9EW73620@earth.backplane.com> 

next in thread | previous in thread | raw e-mail | index | archive | help


On Thu, 11 Jan 2001 12:09:14 PST, Matt Dillon wrote:

>    You should do it right the first time rather then make incremental
>    commits each of which causes pain to the development community.

Ah, okay.

Since I don't see there being any pain caused by this particular commit,
I don't agree with you that we've caused a problem.

I would agree if what we were doing now was going to cause upgrade pain
in the future, but it isn't.  Provided people use mergemaster and follow
the correct upgrade path, folks aren't going to notice what we've done,
except that some people whose boxes used to lock up on startup after a
crash will notice that this doesn't happen any more.

That said, I agree with the principle that generally, things should be
done correctly the first time, especially where interface changes would
be required later by doing things incorrectly in the beginning.

Anyway, we're better off now than we were a day ago.  And when your [1]
mount_mfs(8) patches hit the tree, we'll be even better off.  We'll be
able to remove the rc fallback code that grovels around in /var/db/ and
just use '/var/db' as the default entropy_dir.

Ciao,
Sheldon.

[1] No, this isn't the usual sarcastic criticism of someone's failure to
    do work.  I mean "the patches that you have suggested", and I don't
    really care who submits them.


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




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