svn commit: r316487 - head/etc

[Cy Schubert]

In message <861st64hjz.fsf at desk.des.no>, =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?
= w
rites:
> Cy Schubert <Cy.Schubert at komquats.com> writes:
> > This patch caused some boot failures because the contents of rc_force were 
> > redefined from "yes" (for yes) and NULL (for no) to the words "yes" and 
> > "no". This in turn caused etc/rc.d/dhclient to assume force when $rc_force 
> > was not NULL (test -z failed). Interfaces with static IP addresses invoked 
> > rc.d/dhclient through devd. The attached patch should make all instances 
> > $rc_force consistent with your change.
> 
> Yes, I just realized.  I committed the wrong version of the patch.  I
> had a version in a different tree which added check_force
> etc. functions.
> 
> Could you please revert?  I'd like to spend a little more time making
> sure I get everything right.

Sure, it's been reverted. People will need to remember to not run 
mergemaster or avoid updating rc.subr until the final version is committed 
or dhclient will *always* be started by devd regardless of rc.conf.

Are you thinking of a more intrusive change like my first draft of the 
patch? ngie@ suggested that there be an UPDATING entry.


-- 
Cheers,
Cy Schubert <Cy.Schubert at cschubert.com>
FreeBSD UNIX:  <cy at FreeBSD.org>   Web:  http://www.FreeBSD.org

	The need of the many outweighs the greed of the few.