rlog, cvs2gitdump: Fix handling of revision number greater than one
It exists CVS respositories where revision numbers greather than 1.x are used to version files.
Previous loader implementation was raising an error when encountering such kind of revision so ensure it will be processed as the other ones.
Also fix tag names extraction from rlog output.
Related to #4043 (closed)
Fixes SWH-LOADER-CVS-4 and SWH-LOADER-CVS-6
Test Plan
I added a loading test for a lighweight CVS repo with such revision numbers in its changesets.
I also compared the snapshot obtained with the git cvsimport command and the one obtained by the loader: the exact same set of revisions are generated.
Migrated from D8103 (view on Phabricator)
Merge request reports
Activity
Build is green
Patch application report for D8103 (id=29256)
Rebasing onto e5215cf1...
Current branch diff-target is up to date.
Changes applied before test
commit 94ea5f5ccf1ff76b87d2db36df092711bcd7b1c4 Author: Antoine Lambert <anlambert@softwareheritage.org> Date: Fri Jul 8 18:12:07 2022 +0200 rlog, cvs2gitdump: Fix handling of revision number greater than one It exists CVS respositories where revision numbers greather than 1.x are used to version files. Previous loader implementation was raising an error when encountering such kind of revision so ensure it will be processed as the other ones. Related to #4043
See https://jenkins.softwareheritage.org/job/DLDCVS/job/tests-on-diff/119/ for more details.
Build is green
Patch application report for D8103 (id=29257)
Rebasing onto e5215cf1...
Current branch diff-target is up to date.
Changes applied before test
commit c8525164a0c089f2078921e035d824e9b707f8a9 Author: Antoine Lambert <anlambert@softwareheritage.org> Date: Fri Jul 8 18:12:07 2022 +0200 rlog, cvs2gitdump: Fix handling of revision number greater than one It exists CVS respositories where revision numbers greather than 1.x are used to version files. Previous loader implementation was raising an error when encountering such kind of revision so ensure it will be processed as the other ones. Also fix tag names extraction from rlog output. Related to #4043
See https://jenkins.softwareheritage.org/job/DLDCVS/job/tests-on-diff/120/ for more details.
Build is green
Patch application report for D8103 (id=29258)
Rebasing onto e5215cf1...
Current branch diff-target is up to date.
Changes applied before test
commit ef959a9e6560390cbbc377faabbf2928d3471b62 Author: Antoine Lambert <anlambert@softwareheritage.org> Date: Fri Jul 8 18:12:07 2022 +0200 rlog, cvs2gitdump: Fix handling of revision number greater than one It exists CVS respositories where revision numbers greather than 1.x are used to version files. Previous loader implementation was raising an error when encountering such kind of revision so ensure it will be processed as the other ones. Also fix tag names extraction from rlog output. Related to #4043
See https://jenkins.softwareheritage.org/job/DLDCVS/job/tests-on-diff/121/ for more details.
Build is green
Patch application report for D8103 (id=29259)
Rebasing onto e5215cf1...
Current branch diff-target is up to date.
Changes applied before test
commit 10c4a100f9cb186c3b90fa68edd84673ba089050 Author: Antoine Lambert <anlambert@softwareheritage.org> Date: Fri Jul 8 18:12:07 2022 +0200 rlog, cvs2gitdump: Fix handling of revision number greater than one It exists CVS respositories where revision numbers greather than 1.x are used to version files. Previous loader implementation was raising an error when encountering such kind of revision so ensure it will be processed as the other ones. Also fix tag names extraction from rlog output. Related to #4043
See https://jenkins.softwareheritage.org/job/DLDCVS/job/tests-on-diff/122/ for more details.
Some references in the commit message have been migrated:
- T4043 is now #4043 (closed)
Build is green
Patch application report for D8103 (id=29273)
Rebasing onto 5a79a325...
Current branch diff-target is up to date.
Changes applied before test
commit 386a68f1498fb796b5334a392b100a6e81f9edcc Author: Antoine Lambert <anlambert@softwareheritage.org> Date: Fri Jul 8 18:12:07 2022 +0200 rlog, cvs2gitdump: Fix handling of revision number greater than one It exists CVS respositories where revision numbers greather than 1.x are used to version files. Previous loader implementation was raising an error when encountering such kind of revision so ensure it will be processed as the other ones. Also fix tag names extraction from rlog output. Related to #4043
See https://jenkins.softwareheritage.org/job/DLDCVS/job/tests-on-diff/124/ for more details.