From: Michael Haggerty <mhagger@alum.mit.edu>
To: esr@thyrsus.com
Cc: git@vger.kernel.org
Subject: Re: Change in cvsps maintainership, abd a --fast-export option
Date: Fri, 21 Dec 2012 23:16:29 +0100 [thread overview]
Message-ID: <50D4DFBD.7080708@alum.mit.edu> (raw)
In-Reply-To: <20121221214311.GA29147@thyrsus.com>
On 12/21/2012 10:43 PM, Eric S. Raymond wrote:
> Michael Haggerty <mhagger@alum.mit.edu>:
>> Perhaps your experience is with an older version of cvs2svn?
>
> Well, it has been at least four years since I ran it on anything.
> Maybe that counts as old.
cvs2svn version 2.0 (Aug 2007) totally changed how cvs2svn deduces
changesets. Any results from before that are utterly irrelevant to
modern times.
The changes between version 2.0 and version 2.4 (Sep 2012) have done a
lot more to improve the quality of the conversion. The state of the art
is far beyond what it was four years ago.
>> If not,
>> please be specific rather than just making complaints that are too vague
>> to be rebutted or fixed (whichever is appropriate). I put a *lot* of
>> effort into getting cvs2svn to run correctly, and I take bug reports
>> very seriously.
>
> I can't be specific now, but that may change shortly. I'm putting
> together a test suite for cvsps with the specific intention of
> capturing as many odd corner cases as I can. (I just finished writing
> a small Python framework for expressing interleaved CVS command
> sequences on multiple checkouts in a way that can be easily run.)
>
> It wouldn't be difficult for me to test whether these break cvs2svn.
> You've established that someone over there is paying attention, so
> I'll do that, I guess.
I look forward to your results (whether positive or negative).
> I'm willing to share my test suite as well. Do you have your own zoo
> of odd cases I could test on?
Yes, we have a rather extensive test suite that includes lots of CVS
perversions. See run-tests.py and the CVS repositories under test-data.
Unfortunately, the tests mostly cover only 2svn conversions, but since
most of the tricky work is in the common code that infers the CVS
history, cvs2git benefits from the tests too.
This discussion is getting pretty cvs2svn-specific, so we should
probably move it over to the cvs2svn-dev mailing list.
Michael
--
Michael Haggerty
mhagger@alum.mit.edu
http://softwareswirl.blogspot.com/
next prev parent reply other threads:[~2012-12-21 22:16 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-20 21:56 Change in cvsps maintainership, abd a --fast-export option Eric S. Raymond
2012-12-21 8:11 ` Michael Haggerty
2012-12-21 10:44 ` Eric S. Raymond
2012-12-21 19:19 ` Michael Haggerty
2012-12-21 21:43 ` Eric S. Raymond
2012-12-21 22:16 ` Michael Haggerty [this message]
2012-12-22 3:47 ` Heiko Voigt
2012-12-22 6:21 ` Eric S. Raymond
2012-12-22 13:04 ` Heiko Voigt
2012-12-22 14:15 ` Eric S. Raymond
2012-12-23 19:57 ` Heiko Voigt
2012-12-22 14:04 ` Eric S. Raymond
2012-12-22 14:10 ` Antoine Pelisse
2012-12-22 14:17 ` Eric S. Raymond
2012-12-22 16:25 ` Andreas Schwab
2012-12-22 14:21 ` Andreas Schwab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50D4DFBD.7080708@alum.mit.edu \
--to=mhagger@alum.mit.edu \
--cc=esr@thyrsus.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).