From owner-freebsd-current Thu Jan 29 06:35:20 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id GAA15823 for current-outgoing; Thu, 29 Jan 1998 06:35:20 -0800 (PST) (envelope-from owner-freebsd-current@FreeBSD.ORG) Received: from tecumseh.altavista-software.com (tecumseh.altavista-software.com [205.181.164.14]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id GAA15792 for ; Thu, 29 Jan 1998 06:35:17 -0800 (PST) (envelope-from matt@3am-software.com) Received: from nowin (1Cust109.max11.boston.ma.ms.uu.net [153.35.74.237]) by tecumseh.altavista-software.com (8.8.5/8.8.5) with SMTP id JAA19360; Thu, 29 Jan 1998 09:34:43 -0500 (EST) Message-Id: <199801291434.JAA19360@tecumseh.altavista-software.com> X-Sender: 3ampop@ranier.altavista-software.com X-Mailer: QUALCOMM Windows Eudora Pro Version 4.0 Date: Thu, 29 Jan 1998 09:34:01 -0500 To: Stephen McKay From: Matt Thomas Subject: Re: PATCH: if_de.c #ifdef based version encoding Cc: freebsd-current@FreeBSD.ORG In-Reply-To: <199801291401.AAA12197@troll.dtir.qld.gov.au> References: <199801280243.VAA17410@tecumseh.altavista-software.com> <34CE4438.15FB7483@whistle.com> <199801271101.DAA25275@implode.root.com> <34CE4438.15FB7483@whistle.com> <199801280243.VAA17410@tecumseh.altavista-software.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG X-To-Unsubscribe: mail to majordomo@FreeBSD.org "unsubscribe current" At 09:01 AM 1/29/98 , Stephen McKay wrote: >On Tuesday, 27th January 1998, Matt Thomas wrote: > >[ Regarding version conditionals in if_de.c ] >>>Making Matt's life difficult would be counterproductive, but these version >>>tests are pretty silly from an ivory tower purist view, and are also a pain >>>in practice. I'm with Terry on this one. >> >>If there is such a symbol on 2.1.0 or later I'll use it. > >In the FreeBSD CVS tree, if_de.c is very different in 2.1.x, 2.2.x and 3.0. >Discussions so far imply that you (Matt) intend that your version of if_de.c >should run on all FreeBSD releases. In practice, the code is tweaked for >changing local conditions (ie local to a branch), including mods for stuff >like devconf (deceased) and XNS. I run both 2.1-stable and 2.2-stable so I intend it on both of those (ifdefs and all). Eventually when I get a machine running 3.0 I add the 3.0 changes. >So, I don't think it is a matter of supplying you with a symbol to examine. >The real question is whether or not all FreeBSD branches should be (nearly) >identical (and therefore #ifdef festooned), or whether they should be quite >different, but cleaner. Your call but I'm going to keep the ifdef's. Until there's a clean way of determining 3.0 independent of the compiler, I'm not adding 3.0 ifdef's. -- Matt Thomas Internet: matt@3am-software.com 3am Software Foundry WWW URL: http://www.3am-software.com/bio/matt/ Nashua, NH Disclaimer: I disavow all knowledge of this message