From owner-cvs-all@FreeBSD.ORG Mon Jan 16 15:45:52 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 1F0F816A41F; Mon, 16 Jan 2006 15:45:52 +0000 (GMT) (envelope-from wes@softweyr.com) Received: from softweyr.homeunix.net (cpe-66-75-57-175.san.res.rr.com [66.75.57.175]) by mx1.FreeBSD.org (Postfix) with ESMTP id A3CF343D6A; Mon, 16 Jan 2006 15:45:43 +0000 (GMT) (envelope-from wes@softweyr.com) Received: from [204.68.178.34] (gulliver [204.68.178.34]) (authenticated bits=0) by softweyr.homeunix.net (8.12.10/8.12.10) with ESMTP id k0GFjgdD085900 (version=TLSv1/SSLv3 cipher=EDH-DSS-DES-CBC3-SHA bits=168 verify=NO); Mon, 16 Jan 2006 07:45:42 -0800 (PST) (envelope-from wes@opensail.org) In-Reply-To: <20060116134244.GL53724@submonkey.net> References: <200601152312.k0FNCQOw028323@repoman.freebsd.org> <20060116082403.GC81316@ip.net.ua> <20060116092829.GJ53724@submonkey.net> <20060116100535.GF81316@ip.net.ua> <20060116134244.GL53724@submonkey.net> Mime-Version: 1.0 (Apple Message framework v746.2) Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed Message-Id: Content-Transfer-Encoding: 7bit From: Wes Peters Date: Mon, 16 Jan 2006 07:45:36 -0800 To: Ceri Davies X-Mailer: Apple Mail (2.746.2) Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Ruslan Ermilov , cvs-all@FreeBSD.org Subject: Re: cvs commit: src/etc/rc.d root X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Jan 2006 15:45:52 -0000 On Jan 16, 2006, at 5:42 AM, Ceri Davies wrote: > On Mon, Jan 16, 2006 at 12:05:35PM +0200, Ruslan Ermilov wrote: >> On Mon, Jan 16, 2006 at 09:28:29AM +0000, Ceri Davies wrote: >>> On Mon, Jan 16, 2006 at 10:24:03AM +0200, Ruslan Ermilov wrote: >>>> On Sun, Jan 15, 2006 at 11:12:26PM +0000, Ceri Davies wrote: >>>>> ceri 2006-01-15 23:12:26 UTC >>>>> >>>>> FreeBSD src repository >>>>> >>>>> Modified files: >>>>> etc/rc.d root >>>>> Log: >>>>> Marius Nuennerich pointed out that nextboot(8) configured >>>>> boot options >>>>> were now sticky. This script was deleting /boot/nextkernel >>>>> on boot, but >>>>> there is no code in the tree that creates that file since >>>>> revision 1.15 >>>>> of src/sbin/reboot/reboot.c. >>>>> nextboot(8) creates /boot/nextboot.conf, so remove that instead. >>>>> >>>>> Approved by: jhb (proxy mentor) >>>>> MFC after: 1 week >>>>> >>>>> Revision Changes Path >>>>> 1.12 +2 -2 src/etc/rc.d/root >>>>> >>>> It wasn't sticky -- /boot/support.4th overwrites /boot/ >>>> nextboot.conf >>>> with ``nextboot_enable="NO"''. After boot: >>>> >>>> $ cat /boot/nextboot.conf >>>> nextboot_enable="NO" >>>> kernel="kernel" >>>> kernel_options="" >>> >>> I saw that too. The report I had was that it wasn't doing that >>> however, >>> which is a question for another day. The patch is correct >>> despite this, >>> I believe. >>> >> I'm not happy with implementation either, maybe it should instead run >> "nextboot -D"? > > That makes it dependent on /usr, which would be OK if this wasn't > intended for test booting stuff. All I've done is correct a 4 year > old > oversight (nextkernel was renamed to nextboot.conf and this was > missed). > I'll leave any new implementation to someone else. I'll stick my nose in here as the original implementer of this trivial toy. nextboot.conf is only EVER needed as the result of running nextboot, so removing the file on any normal boot is an entirely reasonable thing to do. Ruslan, I like your idea of 'nextboot -D'. The nextboot utility is written in /bin/sh and located in /sbin, so adding this feature won't make the program any bigger. I'll finish this little project today, unless you've already done it. -- Where am I, and what am I doing in this handbasket? Wes Peters wes@softweyr.com