cvs commit: src/usr.sbin/lpr/lpd lpd.c ...

[David O'Brien]

On Sun, Jul 06, 2003 at 10:52:03PM -0600, M. Warner Losh wrote:
> In message: <20030707040326.GB26228 at dragon.nuxi.com>
>             "David O'Brien" <obrien at FreeBSD.org> writes:
> : On Sun, Jul 06, 2003 at 09:20:37PM -0400, Garance A Drosihn wrote:
> : > Well, any change to use __FBSDID() has to #include <sys/cdefs.h>
> : > to get FBSDID() to work.  Of course, /usr/include/sys/cdefs.h
> : > does not exist on all platforms either (and is certainly not
> : > likely to define __FBSDID even if the file does exist), so that's
> : > just more of an annoyance when I'm doing cross-platform work.
> : 
> : #ifdef __FreeBSD__
> : #include <sys/cdefs.h>
> : __FBSDID("$FreeBSD$");
> : #endif
> 
> #include <sys/cdefs.h>
> #ifdef __FBSDID
> __FBSDID("$FreeBSD$");
> #endif
> 
> might be better...

Not really -- Garance needed to deal with <sys/cdefs.h> not existing
everywhere -- that was one of his key issues.
 
-- 
-- David  (obrien at FreeBSD.org)