At 5:32 PM -0700 7/6/03, Alfred Perlstein wrote:
>* Garance A Drosihn <firstname.lastname@example.org> [030706 16:17] wrote:
>> Speaking only for lpd, I wish you would not have added FBSDID.
>> I had started to add FBSDID to all the source files of
>> lpr & friends, and found that it was just annoying when I
>> then went to build lpr on other platforms. So, I switched
> > back to just plain 'rcsid ='.
>Isn't there a common header where you can just add:
>#define __FBSDID(s) struct __hack
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.
Other than the #include for sys/cdefs.h, there *is* no include
before the reference to __FBSDID in any source file, due to the
standard ordering of lines in a source file.
I remember asking about this with a few people before I decided
to stick with rcsid instead of FBSDID(), but I don't remember
if that was in a mailing list or in private email. It looks
like I had only committed a change to use FBSDID() in one file,
lpr/chkprintcap/skimprintcap.c, and the comment when I switched
that back to rcsid says "discussed with bde and obrien", so it
was probably just private email.
I thought I had switched more source files to use FBSDID, but
it looks like that was the only file where I had made a commit
that I later backed out. I was probably in the middle of an
update to switch all the rest of the files when I started to
realize what an added hassle it was going to be for me.
[ ignoring that question for the moment, why would I want
to define __FBSDID as 'struct __hack'? ]
Yes, there are ways I can get around all of these minor issues,
but at the time it seemed like a lot of extra futzing around
for very little gain.
Garance Alistair Drosehn = email@example.com
Senior Systems Programmer or firstname.lastname@example.org
Rensselaer Polytechnic Institute or email@example.com