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

[Bruce Evans]

On Tue, 11 May 2004, John Baldwin wrote:

> On Tuesday 11 May 2004 03:23 pm, Nate Lawson wrote:
> > On Tue, 11 May 2004, Colin Percival wrote:
> > > ...
> > > 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; 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.
> >
> > I agree.  This creates the same problems as panic() now changing object
> > files every time a comment is moved or other non-functional change.
>
> Seconded on both counts.

This is not as bad as the programs that use __DATE__ or __TIME__, or
generate a version.c file whose version changes on every recompile, so
that repeated buildworlds cannot produce the same result twice.  I'd
like to fix these, but most of them are contribware.  There are also
timestamps in *.a, but these files are easier to ignore.

Bruce