Subject: Re: CVS commit: basesrc/bin/pax
To: Christos Zoulas <christos@zoulas.com>
From: Todd Vierling <tv@wasabisystems.com>
List: source-changes
Date: 01/31/2002 16:54:58
On Thu, 31 Jan 2002, Christos Zoulas wrote:

: >Modified Files:
: >	basesrc/bin/pax: ar_io.c ar_subs.c buf_subs.c cpio.c file_subs.c
: >	    gen_subs.c pat_rep.c pax.c sel_subs.c tables.c tar.c tty_subs.c
: >
: >Log Message:
: >Protect __RCSID and __COPYRIGHT from being invoked if not defined.

: I don't see the point of that since we are including *our* <sys/cdefs.h>

Not for a host tool.  In fact, we *CAN'T* include "our" <sys/cdefs.h> for
using pax as a host tool, since a foreign host may have its own
<sys/cdefs.h> that defines things needed for that host.

Of course, if KNF didn't specify direct inclusion of <sys/cdefs.h> at the
very top of the file (which it shouldn't, braindead bull$%^t), I wouldn't
have to fight this bogosity.

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