In message <861st64hjz....@desk.des.no>, =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav? = w rites: > Cy Schubert <cy.schub...@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.schub...@cschubert.com> FreeBSD UNIX: <c...@freebsd.org> Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few. _______________________________________________ svn-src-all@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
