Subject: Re: CVS commit: basesrc/bin/pax
To: Perry E. Metzger <perry@wasabisystems.com>
From: Todd Vierling <tv@wasabisystems.com>
List: source-changes
Date: 02/01/2002 10:06:18
On 1 Feb 2002, Perry E. Metzger wrote:

: > : Maybe we should define an #include file that only does _RCSID and
: > : _COPYRIGHT?
: >
: > What I was trying to say is that if the __RCSID and __COPYRIGHT macros
: > lived *below* the include list rather than above it,
:
: I think that would be bad, but I suggest that a single
: "#include <ids.h>" or some similar thing would be good, so we can make
: it more portable. One wants the RCSID as close to the top as possible.

Then make sure it's as non-clashing with other common include file names as
possible.

Where __RCSID is located in the source file is not relevant, contrary to
what you imply here.  It just ends up in the .o file as a string, and
there's already a rcsid in a C comment at the top of each file.  Mandating
that __RCSID immediately follow the copyright blurb doesn't buy you squat.

In any case, I've already fixed the instances used by host tools in what I
found to be the least intrusive way possible.  All of the above changes
would be a bit more intrusive into the files.  You can feel free to invent
another way, if you wish.  8-)

-- 
-- Todd Vierling <tv@wasabisystems.com>  *  Wasabi & NetBSD:  Run with it.
-- CDs, Integration, Embedding, Support -- http://www.wasabisystems.com/