From owner-cvs-all Fri May 3 18:31:47 2002 Delivered-To: cvs-all@freebsd.org Received: from fw.wemm.org (12-232-135-171.client.attbi.com [12.232.135.171]) by hub.freebsd.org (Postfix) with ESMTP id E9FD037B400; Fri, 3 May 2002 18:31:40 -0700 (PDT) Received: from overcee.wemm.org (overcee.wemm.org [10.0.0.3]) by fw.wemm.org (8.11.6/8.11.6) with ESMTP id g441Ve412219; Fri, 3 May 2002 18:31:40 -0700 (PDT) (envelope-from peter@wemm.org) Received: from wemm.org (localhost [127.0.0.1]) by overcee.wemm.org (Postfix) with ESMTP id 64FF538FD; Fri, 3 May 2002 18:31:40 -0700 (PDT) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Bruce Evans Cc: "J. Mallett" , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.bin/xargs xargs.c In-Reply-To: <20020504052140.T8741-100000@gamplex.bde.org> Date: Fri, 03 May 2002 18:31:40 -0700 From: Peter Wemm Message-Id: <20020504013140.64FF538FD@overcee.wemm.org> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Bruce Evans wrote: > On Thu, 2 May 2002, J. Mallett wrote: > > > On Fri, May 03, 2002 at 07:30:44AM +1000, Bruce Evans wrote: > > > On Wed, 1 May 2002, J. Mallett wrote: > > > > > > > jmallett 2002/05/01 22:21:35 PDT > > > > > > > > Modified files: > > > > usr.bin/xargs xargs.c > > > > Log: > > > > __COPYRIGHT() and __SCCSID(). > > > > > > These should never be used. They create conflicts for future imports > > > and enlarge diffs with the vendor version by editing vendor lines, > > > and using __COPYRIGHT demonstrates a bug in its implementation: > > > > Sorry about this. I'll back out SCCSID and COPYRIGHT usage here, as well > > as m4(1), tomorrow. > > Sorry to pick on you. > > Perhaps a better example of messing with the vendor copyright string is in > login/login.c. The string was first commented out and replaced by a > quite different string generated by , and later the > commented out code was removed. IMO, is only right for > the kernel, but since it is only used in one place in the kernel, it > shouldn't exist as a separate header. would be better > if it added to existing copyright strings instead of replacing them. It just so happens that I got burned by this last night with an early buildworld failure. Cheers, -Peter -- Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com "All of this is for nothing if we don't go to the stars" - JMS/B5 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message