Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 03 May 2002 18:31:40 -0700
From:      Peter Wemm <peter@wemm.org>
To:        Bruce Evans <bde@zeta.org.au>
Cc:        "J. Mallett" <jmallett@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/usr.bin/xargs xargs.c 
Message-ID:  <20020504013140.64FF538FD@overcee.wemm.org>
In-Reply-To: <20020504052140.T8741-100000@gamplex.bde.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
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 <sys/copyright.h>, and later the
> commented out code was removed.  IMO, <sys/copyright> 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.  <sys/copyright.h> 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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020504013140.64FF538FD>