From owner-cvs-all Thu Jul 8 10:13:46 1999 Delivered-To: cvs-all@freebsd.org Received: from apollo.backplane.com (apollo.backplane.com [209.157.86.2]) by hub.freebsd.org (Postfix) with ESMTP id E612D14C31; Thu, 8 Jul 1999 10:13:40 -0700 (PDT) (envelope-from dillon@apollo.backplane.com) Received: (from dillon@localhost) by apollo.backplane.com (8.9.3/8.9.1) id KAA41266; Thu, 8 Jul 1999 10:13:30 -0700 (PDT) (envelope-from dillon) Date: Thu, 8 Jul 1999 10:13:30 -0700 (PDT) From: Matthew Dillon Message-Id: <199907081713.KAA41266@apollo.backplane.com> To: "Rodney W. Grimes" Cc: peter@netplex.com.au (Peter Wemm), cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/etc rc.network src/etc/defaults rc.conf References: <199907081701.KAA41342@gndrsh.aac.dev.com> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk I really like Rod's idea better. By putting: network_interfaces="`ifconfig -l`" ^^^^ note: that should be /sbin/ifconfig, because /sbin may not be in the path in all cases. Into /etc/defaults/rc.conf we would maintain virtually complete backwards compatibility while getting the same result. It is the most painless way to make this change, and people can still override it in /etc/rc.conf. This is a whole lot better then making a change that ignores the network_interface variable. -Matt Matthew Dillon :> > > Modified files: :> > > etc rc.network :> > > etc/defaults rc.conf :> > > Log: :> > > Do away with ${network_interfaces} in rc.conf. Just use `ifconfig -l` to :> > > get a list of interfaces, and then automatically configure them if :> > > ${ifconfig_${ifn}} or /etc/start_if.${ifn} exists. :> > > :> > > This makes it a lot easier to deal with machines that constantly change :> > > their network configuration as you can leave ifconfig settings for all :> > > the possible cards - just the ones that are present will be configured. :> > > :> > > Revision Changes Path :> > > 1.48 +12 -3 src/etc/rc.network :> > > 1.16 +1 -2 src/etc/defaults/rc.conf :> > :> > I strongly dislike this idea. If I list an interface in network_interfaces :> > during boot: :> > :> > a) I want to see an error if the physical device is missing :> > :> > b) It might not even exist in an ifconfig -l at this time, it :> > just might be getting created by /etc/start_if.X :> > :> > IMHO trying to make this smart is going to lead you down the road to :> > the problems that happen on Micro$oft systems. :> :> The original version I did merged ifconfig -l with ${network_interfaces) :> and suppressed duplicates. That would have generated diagnostics on a :> missing interface. I ended up leaving it out since all our existing :> interfaces appear at boot time, even if they have not been used. It seemed :> a bit of overkill, but if people want this, I'm happy to revive that... : :I don't think that would be much better. I've shown you how to make :the current (before your commit) system do what you want, and still do :what I want. What more do you need? : :network_interfaces="`ifconfig -l`" : :... :had back without major surgery... :-- :Rod Grimes - KD7CAX - (RWG25) rgrimes@gndrsh.dnsmgr.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message