From owner-freebsd-bugs Fri Sep 7 15:40:19 2001 Delivered-To: freebsd-bugs@hub.freebsd.org Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 72DD237B405 for ; Fri, 7 Sep 2001 15:40:01 -0700 (PDT) Received: (from gnats@localhost) by freefall.freebsd.org (8.11.4/8.11.4) id f87Me1j54439; Fri, 7 Sep 2001 15:40:01 -0700 (PDT) (envelope-from gnats) Received: from mailsrv.otenet.gr (mailsrv.otenet.gr [195.170.0.5]) by hub.freebsd.org (Postfix) with ESMTP id 7C57437B407 for ; Fri, 7 Sep 2001 15:30:27 -0700 (PDT) Received: from hades.hell.gr (patr530-b096.otenet.gr [195.167.121.224]) by mailsrv.otenet.gr (8.11.5/8.11.5) with ESMTP id f87MUNA16383 for ; Sat, 8 Sep 2001 01:30:23 +0300 (EEST) Received: (from charon@localhost) by hades.hell.gr (8.11.6/8.11.6) id f87Ljqe13039; Sat, 8 Sep 2001 00:45:52 +0300 (EEST) (envelope-from charon) Message-Id: <200109072145.f87Ljqe13039@hades.hell.gr> Date: Sat, 8 Sep 2001 00:45:52 +0300 (EEST) From: Giorgos Keramidas Reply-To: Giorgos Keramidas To: FreeBSD-gnats-submit@freebsd.org X-Send-Pr-Version: 3.113 Subject: misc/30423: CVSROOT/modules has mistakenly single-line comments in some module-aliases Sender: owner-freebsd-bugs@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org >Number: 30423 >Category: misc >Synopsis: CVSROOT/modules has mistakenly single-line comments in some module-aliases >Confidential: no >Severity: serious >Priority: medium >Responsible: freebsd-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: change-request >Submitter-Id: current-users >Arrival-Date: Fri Sep 07 15:40:01 PDT 2001 >Closed-Date: >Last-Modified: >Originator: Giorgos Keramidas >Release: FreeBSD 5.0-CURRENT i386 >Organization: >Environment: System: FreeBSD hades.hell.gr 5.0-CURRENT FreeBSD 5.0-CURRENT #5: Tue Sep 4 19:49:44 EEST 2001 root@hades.hell.gr:/usr/obj/usr/src/sys/CHARON i386 >Description: When using 'cvs co -c' on /home/ncvs (my local CVS mirror), I saw that a few lines were not actual module aliases, but plain empty lines, starting with whitespace and '#' characters. Looking at CVSROOT/modules, I found lines like: -------------------------------------------------------------- bsd_apropos src/usr.bin/apropos #deleted files -------------------------------------------------------------- The info page of CVS includes a detailed description of 'module options' that can go after the module name in a line, but does not mention 'single line comments'. It only mentions 'comment lines' in the following context (in the section: Top -> Administrative files -> modules). The `modules' file may contain blank lines and comments (lines beginning with `#') as well as module definitions. >How-To-Repeat: The lines that include 'single line comments' in CVSROOT/modules cause 'cvs co -c' to list fake module entries like the ones shown below: % cvs -R -d /home/ncvs co -c | grep '^[^a-zA-Z0-9]' #deleted files ... This is obviously not a real module in the repository. And it is not a comment line either. >Fix: The attached patch removes the comments from those lines of CVSROOT/modules that contain such comment-like parts. Of course, instead of removing these lines altogether, one might prefer to move them `above' the relevant module lines. --- patch.diff begins here --- Index: modules =================================================================== RCS file: /home/ncvs/CVSROOT/modules,v retrieving revision 1.3817 diff -u -u -r1.3817 modules --- modules 6 Sep 2001 22:39:22 -0000 1.3817 +++ modules 7 Sep 2001 21:35:43 -0000 @@ -534,15 +534,15 @@ brouted ports/net/brouted brs ports/misc/brs bs src/games/bs -bsd_apropos src/usr.bin/apropos #deleted files -bsd_diff src/usr.bin/diff #just man pages -bsd_grep src/usr.bin/grep #appears to be just egrep? -bsd_ld src/usr.bin/ld #deleted files -bsd_man src/usr.bin/man #deleted files -bsd_patch src/usr.bin/patch #deleted obsolete files -bsd_sort src/usr.bin/sort #just man pages +bsd_apropos src/usr.bin/apropos +bsd_diff src/usr.bin/diff +bsd_grep src/usr.bin/grep +bsd_ld src/usr.bin/ld +bsd_man src/usr.bin/man +bsd_patch src/usr.bin/patch +bsd_sort src/usr.bin/sort bsd_uucp src/usr.bin/uucp -bsd_whatis src/usr.bin/whatis #deleted files +bsd_whatis src/usr.bin/whatis bsdi-netscape47-communicator ports/www/bsdi-netscape47-communicator bsdi-netscape47-navigator ports/www/bsdi-netscape47-navigator bsmtp ports/mail/bsmtp @@ -1924,7 +1924,7 @@ gzip src/gnu/usr.bin/gzip h2n ports/net/h2n ha ports/archivers/ha -hack src/games/hack #should be replaced by nethack! +hack src/games/hack hagelslag ports/net/hagelslag halflifeserver ports/games/halflifeserver handbook doc/en_US.ISO8859-1/books/handbook --- patch.diff ends here --- >Release-Note: >Audit-Trail: >Unformatted: To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message