Skip to content

rlog: fix loading of CVS commits which have a commit ID

Stefan Sperling requested to merge generated-differential-D6561-source into generated-differential-D6561-target

The CVS commit ID is an optional attribute which is only generated by relatively recent releases of CVS clients. Our rlog parser was skipping such commits because it failed to match on them due to an error in a regular expression. This resulted in an incomplete import of CVS revision history.

Here is a sample line from cvs rlog output which carries a commit ID and was not matched because the regex lacked the trailing semicolon: date: 2007-07-17 15:02:50 +0200; author: larsl; state: Exp; lines: +619 -285; commitid: oju0x8tTc9aUB7qs;

Found while testing ingestion of the GNU dino repository from cvs.sannah.gnu.org/sources/dino

Migrated from D6561 (view on Phabricator)

Merge request reports