From owner-cvs-src@FreeBSD.ORG Wed Feb 18 09:52:55 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A0EF916A4CE for ; Wed, 18 Feb 2004 09:52:55 -0800 (PST) Received: from root.org (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 46A9943D1F for ; Wed, 18 Feb 2004 09:52:55 -0800 (PST) (envelope-from nate@root.org) Received: (qmail 35790 invoked by uid 1000); 18 Feb 2004 17:52:55 -0000 Date: Wed, 18 Feb 2004 09:52:55 -0800 (PST) From: Nate Lawson To: Ruslan Ermilov In-Reply-To: <20040218110713.GG54588@FreeBSD.org.ua> Message-ID: <20040218094753.Q35672@root.org> References: <200402170343.i1H3hCot079054@repoman.freebsd.org> <20040218012226.GA16022@nagual.pp.ru> <20040218110713.GG54588@FreeBSD.org.ua> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: Andrey Chernov cc: src-committers@FreeBSD.org cc: cvs-src@FreeBSD.org Subject: Re: cvs commit: src/contrib/groff FREEBSD-Xlist src/contrib/groff/src/include getopt.h src/contrib/groff/src/libs/libgroff getopt.c getopt1.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Feb 2004 17:52:55 -0000 On Wed, 18 Feb 2004, Ruslan Ermilov wrote: > On Wed, Feb 18, 2004 at 07:02:53AM +0100, Ollivier Robert wrote: > > According to Andrey Chernov: > > > > According to Andrey A. Chernov: > > > > > 1.3 +2 -0 src/contrib/groff/FREEBSD-Xlist > > > > > 1.2 +0 -169 src/contrib/groff/src/include/getopt.h (dead) > > > > > 1.2 +0 -1055 src/contrib/groff/src/libs/libgroff/getopt.c (dead) > > > > > 1.2 +0 -188 src/contrib/groff/src/libs/libgroff/getopt1.c (dead) > > > > > that it will be replacement for gnu getopt (as for fnmatch, stpcpy etc gnu > > > pollution). getopt_long() was too long in the libc to really trigger the > > > switch now. I don't take files off the branch, just remove unneded junk, > > > most of it is already in FREEBSD-Xlist. It always be our style to not > > > import unneeded files. > > > > Look at the commit message, these files were on the FSF vendor branch, you > > have taken these off that branch! That's _not_ the way you should have done > > it. > > Removing files on the HEAD branch is somewhat rather special way > to "take files off the vendor branch", and as Andrey already > pointed out, we needed to remove at least getopt.h so the > FreeBSD's native version of getopt.h gets used. And there was > no point keeping other getopt*.c either with this change. This is not the appropriate way to do it. des@, roberto@, and myself have all explained this both times this has happened. -Nate