#150: cvs (via cvsps) fails to parse INITIAL as revision
---------------------+------------------------------------------------------
Reporter: shadow | Owner: lele
Type: defect | Status: new
Priority: major | Milestone: VersionOne
Component: cvsps | Version: 0.9
Resolution: | Keywords:
---------------------+------------------------------------------------------
Changes (by lele):
* component: tailor => cvsps
Old description:
> File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvsps.py",
> line 257, in _getUpstreamChangesets
> for cs in changesets_from_cvsps(log, sincerev):
> File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvsps.py",
> line 191, in changesets_from_cvsps
> if compare_cvs_revs(e.old_revision, fromrev)>0:
> File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvs.py",
> line 53, in compare_cvs_revs
> r1 = normalize_cvs_rev(revstr1)
> File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvs.py",
> line 39, in normalize_cvs_rev
> r = [int(n) for n in rev.split('.')]
> ValueError: invalid literal for int(): INITIAL
>
> looks like fromrev is INITIAL and that just gets passed to the cvs rev
> comparison, which isn't going to work.
>
> What *should* happen?
New description:
{{{
File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvsps.py",
line 257, in _getUpstreamChangesets
for cs in changesets_from_cvsps(log, sincerev):
File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvsps.py",
line 191, in changesets_from_cvsps
if compare_cvs_revs(e.old_revision, fromrev)>0:
File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvs.py",
line 53, in compare_cvs_revs
r1 = normalize_cvs_rev(revstr1)
File "/opt/local/lib/python2.4/site-packages/vcpx/repository/cvs.py",
line 39, in normalize_cvs_rev
r = [int(n) for n in rev.split('.')]
ValueError: invalid literal for int(): INITIAL
}}}
looks like fromrev is INITIAL and that just gets passed to the cvs rev
comparison, which isn't going to work.
What *should* happen?
--
--
Ticket URL: <http://progetti.arstecnica.it/tailor/ticket/150#comment:1>
Tailor <http://progetti.arstecnica.it/tailor>
An inter-VCs changeset exchanger
_______________________________________________
Tailor mailing list
[email protected]
http://lists.zooko.com/mailman/listinfo/tailor
