From owner-cvs-all@FreeBSD.ORG Wed Sep 5 12:57:00 2007 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3F5FD16A417; Wed, 5 Sep 2007 12:57:00 +0000 (UTC) (envelope-from deischen@freebsd.org) Received: from mail.netplex.net (mail.netplex.net [204.213.176.10]) by mx1.freebsd.org (Postfix) with ESMTP id CF04B13C465; Wed, 5 Sep 2007 12:56:59 +0000 (UTC) (envelope-from deischen@freebsd.org) Received: from sea.ntplx.net (sea.ntplx.net [204.213.176.11]) by mail.netplex.net (8.14.1/8.14.1/NETPLEX) with ESMTP id l85CuPGt004359; Wed, 5 Sep 2007 08:56:25 -0400 (EDT) X-Virus-Scanned: by AMaViS and Clam AntiVirus (mail.netplex.net) X-Greylist: Message whitelisted by DRAC access database, not delayed by milter-greylist-3.0 (mail.netplex.net [204.213.176.10]); Wed, 05 Sep 2007 08:56:26 -0400 (EDT) Date: Wed, 5 Sep 2007 08:56:25 -0400 (EDT) From: Daniel Eischen X-X-Sender: eischen@sea.ntplx.net To: Yar Tikhiy In-Reply-To: <20070904122356.GN30502@comp.chem.msu.su> Message-ID: References: <200708270850.20904.jhb@freebsd.org> <200708281142.07941.jhb@freebsd.org> <200708281403.05931.jhb@freebsd.org> <20070829073011.GD598@comp.chem.msu.su> <20070830061935.GF31948@comp.chem.msu.su> <20070904111400.GM30502@comp.chem.msu.su> <20070904122356.GN30502@comp.chem.msu.su> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: cvs-src@freebsd.org, cvs-all@freebsd.org, src-committers@freebsd.org, "M. Warner Losh" , John Baldwin Subject: Re: cvs commit: src/lib/libc/gen fts-compat.c fts-compat.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Daniel Eischen List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 Sep 2007 12:57:00 -0000 On Tue, 4 Sep 2007, Yar Tikhiy wrote: > > I must withdraw my argument against alias versions. My problem > with them appeared to stem from a typo in "__weak_reference" I'd > inherited from your example. :-) > It's funny that the compiler emitted just a vague warning: > > foo.c:12: warning: data definition has no type or storage class > foo.c:12: warning: parameter names (without types) in function declaration > > and the typo remained unnoticed until I saw it by accident. > > Now I'll investigate if macros can help us to have right versions > in branches and HEAD... Do you think there is a non-hackish way > to see if we're on a STABLE branch? __FreeBSD_version doesn't seem > to provide the information until after the first release from the > branch has been cut. I can't think of anything. The other problem is that we may need to MFC an ABI change before -current is branched. When that happens, the RELENG_X version (FBSD_1.?) also needs to exist in -current. Other ABI changes in -current should remain in the hidden or temporary (or whatever we call it) version. I'm still unconvinced that this method is worth the effort or confusion level ;-) > OTOH, we should't dump ObsoleteVersions completely because we may > need it, too, e.g., when we didn't plan to break an ABI twice in a > row but have to. OK, but I suppose you could just create the file when it is needed for the first time. -- DE