cvs commit: src/sys/dev/led led.h

David O'Brien obrien at FreeBSD.org
Tue May 11 12:12:40 PDT 2004


On Tue, May 11, 2004 at 07:54:58PM +0100, Colin Percival wrote:
> At 19:15 11/05/2004, David O'Brien wrote:
> >On Tue, May 11, 2004 at 10:02:59AM -0700, Nate Lawson wrote:
> >> Do we even want __FBSDID in .h files?  It doesn't seem to make sense since
> >> it's not a compilation unit (i.e. no linkage).
> >
> >It actually does make sense, as a header's contents does wind up in a .o
> >eventually :-)  And header contents can be the cause of problems as much
> >as a .c file.  There is no problem having multiple __FBSDID in either
> >a.out or ELF objects.
> 
> However, there are problems with having a header file's __FBSDID tags end
> up in a several binaries.  Quite apart from the resulting bloat, most changes
> to header files don't actually result in many binaries being modified;

As much as any source change.  Style changes don't but changing any
macro's or #define values would produce a modified binary.

> including compilable $Id$ tags in the headers would result in lots of
> spurious binary changes.  This would make me (and anyone who uses FreeBSD
> Update) very unhappy.

But makes developers very happy in that they can find out from bug
reporters the exact revisions of files involved. :-)
 
-- 
-- David  (obrien at FreeBSD.org)


More information about the cvs-all mailing list