From owner-cvs-all Tue Mar 13 12:45:46 2001 Delivered-To: cvs-all@freebsd.org Received: from winston.osd.bsdi.com (winston.osd.bsdi.com [204.216.27.229]) by hub.freebsd.org (Postfix) with ESMTP id E16DE37B71F; Tue, 13 Mar 2001 12:45:38 -0800 (PST) (envelope-from jkh@osd.bsdi.com) Received: from localhost (jkh@localhost [127.0.0.1]) by winston.osd.bsdi.com (8.11.2/8.11.1) with ESMTP id f2DKiBH47798; Tue, 13 Mar 2001 12:44:14 -0800 (PST) (envelope-from jkh@osd.bsdi.com) To: brian@Awfulhak.org Cc: jkh@FreeBSD.org, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/sysinstall config.c In-Reply-To: <200103130857.f2D8vOS30648@hak.lan.Awfulhak.org> References: <200103130857.f2D8vOS30648@hak.lan.Awfulhak.org> X-Mailer: Mew version 1.94.1 on Emacs 20.7 / Mule 4.0 (HANANOEN) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-Id: <20010313124411W.jkh@osd.bsdi.com> Date: Tue, 13 Mar 2001 12:44:11 -0800 From: Jordan Hubbard X-Dispatcher: imput version 20000228(IM140) Lines: 20 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG > At risk of sounding foolish because I didn't look at the diffs.... > > If you're sorting the file, are you potentially re-ordering duplicate > variable assignments, making the final answer different ? And if so, > do you really care ? This only occurs when the file is first created, not later when the possibility of (non-identical) duplicate assignments arises. If that were to happen, I most certainly would care since "YES" sorts later than "NO" and that could potentially be very bad! :-) That's why I only do it once. After that, the file is free to get as silly-looking as it likes and it probably will. The real solution to this problem is to write an rc.conf parser which sucks in all the variables, does the override checking and then spits them back out again in proper shape. There's already an rc.conf parser in sysinstall so it wouldn't actually be that hard, but this seemed like a less risky fix for now. I'll probably go commit something to -current which does it right after 4.3 comes out. - Jordan To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message