Merge lp://qastaging/~brook/bzr-cvsps-import/fix-402690-788914-788915 into lp://qastaging/bzr-cvsps-import

Proposed by Brook Stevens
Status: Merged
Approved by: John A Meinel
Approved revision: 69
Merged at revision: 69
Proposed branch: lp://qastaging/~brook/bzr-cvsps-import/fix-402690-788914-788915
Merge into: lp://qastaging/bzr-cvsps-import
Diff against target: 62 lines (+16/-4)
1 file modified
cvsps/importer.py (+16/-4)
To merge this branch: bzr merge lp://qastaging/~brook/bzr-cvsps-import/fix-402690-788914-788915
Reviewer Review Type Date Requested Status
Bazaar Developers Pending
Review via email: mp+62587@code.qastaging.launchpad.net

Description of the change

Fixes 402690, 788914 and 788915. Also when a commit message fails it will log a warning message rather than failing. For converting a large repository where it may take over a day to run this was desirable to me and maybe others will find it useful.
Tested using the following configuration:
Bazaar (bzr) 2.3.1
  Python interpreter: /usr/bin/python 2.7.1
  Python standard library: /usr/lib/python2.7
  Platform: Linux-2.6.38-8-virtual-x86_64-with-Ubuntu-11.04-natty
  bzrlib: /usr/lib/python2.7/dist-packages/bzrlib
  Bazaar configuration: /home/ubuntu/.bazaar
  Bazaar log file: /home/ubuntu/.bzr.log

To post a comment you must log in.
Revision history for this message
John A Meinel (jameinel) wrote :

I'm fine landing this, though generally you should look more at "bzr-fastimport" and the cvs2bzr toolchain. (part of the cvs2svn family, IIRC).

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: