cvsview2.cgi problems due to new cvs version

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
2 years ago

People

(Reporter: kairo, Assigned: timeless)

Tracking

Trunk
x86
Linux

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

13 years ago
Newer CVS versions add commitid identifiers into their RCS files, and cvsview2.cgi chokes on those, just displaying error messages.

Example of what I see on a directory:
-------
Unexpected parsing error in RCS file .
Expected token: date, but saw: 26904377525b4567
-------

Example of a single file:
-------
Unexpected parsing error in RCS file /usr/local/cvsroot/com/php/include/util/com_database.php-class,v.
Expected token: date, but saw: 5c7943663c364567
-------

I'm seeing this on a private bonsai installation combined with
Concurrent Versions System (CVS) 1.12.12 (client/server)

If someone wants to check this stuff in detail, sp. writing a patch for it, I can give some insight like the URL where I'm seeing this, maybe the RCS file for that file above privately, I just don't want to link this installation to the wide public.
(Reporter)

Comment 1

13 years ago
cvslog.cgi shows the same error, cvsblame.cgi displays a "500 Server Error", I guess the same problem is hidden behind it.
(Assignee)

Comment 2

13 years ago
if you could upload a random ,v file from 1.12 with a few records, that'd be helpful (create it just for this purpose...)
(Reporter)

Comment 3

13 years ago
Created attachment 202924 [details]
robots.txt,v - this has been created with older CVS and updated with 1.12.12

OK, I'm attaching two very small example ,v files here. Both are robots.txt files with the same content, so they should be easy to view :)
This first one has been created with an older CVS version and now I've done an empty forced checkin with the new version that adds this "commitid".
Everyone who upgrades CVS some time will end up with such a mixture...
(Reporter)

Comment 4

13 years ago
Created attachment 202927 [details]
robots2.txt,v - this has been created and updated with CVS 1.12.12

This second file (robots2.txt,v) has been created and updated with the newer CVS version.
(Note that it was not part of the initial import of the dir but added later, that why the 1.1.1.1 stuff might be missing...)
(Reporter)

Comment 5

13 years ago
just as a side note, the second file is from a different repository as well, if you might spot further unrelated inconsistencies between the files :)
(Assignee)

Comment 6

13 years ago
Created attachment 202960 [details] [diff] [review]
handle commitid
Assignee: tara → timeless
Status: NEW → ASSIGNED
Attachment #202960 - Flags: review?(kairo)
(Reporter)

Comment 7

13 years ago
Comment on attachment 202960 [details] [diff] [review]
handle commitid

thanks, this works like a charm on my installation :)
Attachment #202960 - Flags: review?(kairo) → review+
(Assignee)

Comment 8

13 years ago
Comment on attachment 202960 [details] [diff] [review]
handle commitid

mozilla/webtools/bonsai/cvsblame.pl 	1.16
Attachment #202960 - Attachment is obsolete: true
(Assignee)

Updated

13 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.