From owner-cvs-user  Sun Sep 22 22:41:23 1996
Return-Path: owner-cvs-user
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.5/8.7.3) id WAA13678
          for cvs-user-outgoing; Sun, 22 Sep 1996 22:41:23 -0700 (PDT)
Received: from spinner.DIALix.COM (root@spinner.DIALix.COM [192.203.228.67])
          by freefall.freebsd.org (8.7.5/8.7.3) with ESMTP id WAA13511;
          Sun, 22 Sep 1996 22:41:05 -0700 (PDT)
Received: from spinner.DIALix.COM (peter@localhost.DIALix.oz.au [127.0.0.1])
          by spinner.DIALix.COM (8.7.6/8.7.3) with ESMTP id NAA12996;
          Mon, 23 Sep 1996 13:40:49 +0800 (WST)
Message-Id: <199609230540.NAA12996@spinner.DIALix.COM>
X-Mailer: exmh version 1.6.7 5/3/96
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 ...
In-reply-to: Your message of "Sun, 22 Sep 1996 14:57:02 MST."
             <199609222157.OAA05513@freefall.freebsd.org> 
Date: Mon, 23 Sep 1996 13:40:49 +0800
From: Peter Wemm <peter@spinner.DIALix.COM>
Sender: owner-cvs-user@FreeBSD.ORG
X-Loop: FreeBSD.org
Precedence: bulk

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