Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 23 Sep 1996 13:40:49 +0800
From:      Peter Wemm <peter@spinner.DIALix.COM>
To:        Wolfram Schneider <wosch@freefall.freebsd.org>
Cc:        CVS-committers@freefall.freebsd.org, cvs-all@freefall.freebsd.org, cvs-bin@freefall.freebsd.org, cvs-user@freefall.freebsd.org, cvs-libexec@freefall.freebsd.org
Subject:   Re: cvs commit: src/bin Makefile.inc ...
Message-ID:  <199609230540.NAA12996@spinner.DIALix.COM>
In-Reply-To: Your message of "Sun, 22 Sep 1996 14:57:02 MST." <199609222157.OAA05513@freefall.freebsd.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
Wolfram Schneider wrote:
>   Log:
>   add forgotten $Id$
>
>   1.2       +2 -0      src/libexec/atrun/LEGAL
>   1.2       +3 -1      src/libexec/bootpd/Announce
>   1.2       +49 -0     src/libexec/bootpd/Changes
>   1.2       +27 -7     src/libexec/bootpd/Makefile.UNIX
>   1.2       +19 -0     src/libexec/bootpd/Problems
>   1.2       +3 -0      src/libexec/bootpd/README
>   1.2       +4 -5      src/libexec/bootpd/bootptab.mcs
>   1.2       +3 -2      src/libexec/bootpd/dovend.c
>   1.2       +2 -0      src/libexec/bootpd/dumptab.c
>   1.2       +2 -4      src/libexec/bootpd/hash.c
>   1.2       +7 -2      src/libexec/bootpd/hwaddr.h
>   1.2       +5 -0      src/libexec/bootpd/lookup.c
>   1.2       +7 -2      src/libexec/bootpd/patchlevel.h
>   1.2       +10 -19    src/libexec/bootpd/readfile.c
>   1.2       +10 -1     src/libexec/bootpd/trygetea.c
>   1.2       +8 -2      src/libexec/bootpd/trygetif.c
>   1.2       +12 -4     src/libexec/bootpd/trylook.c
>   1.2       +3 -7      src/libexec/bootpd/tools/bootpef/bootpef.c
>   1.2       +25 -5     src/libexec/bootpd/tools/bootptest/bootptest.c
>   1.2       +9 -8      src/libexec/bootpd/tools/bootptest/print-bootp.c

Say What?!??

The files Announce, Changes, Makefile.UNIX, Problems, README, 
bootptab.mcs, dovend.c, dumptab.c, hash.c, hwaddr.h, lookup.c, 
patchlevel.h, readfile.c, trytea.c, trygetif.c, trylook.c, bootpef.c, 
bootptest.c, and print-bootp.c should ***NOT*** have $Id$ because they 
have not been locally modified!

The unusual +10 -19 markers are harmless, it is an artifact of the way RCS 
files are stored.  The 1.2 delta is relative the oldest version imported, 
not the newest as one would expect.  ie: the 1.1.1.1 -> 1.2 delta is 
largely a duplicate of the 1.1.1.1 -> 1.1.1.2 delta.

This has just unnecessarily caused any future updates to bootpd to be 
double the normal size. :-(  Committing to files that are currently on the 
vendor branch just to add $Id$ is a major *NO*!  It's a technical issue 
that's not easily fixed.  Yes, it's a major flaw in RCS/CVS, which is one 
major reason for the start-again approach in contrib - to get out of this 
mess in the gnu/* tree.  This problem will persist as long as we stick 
with cvs or any other rcs-based tool.  (Incidently, p3 doesn't have this 
problem.)

Sigh,
-Peter





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199609230540.NAA12996>