login
Header Space

 
 

Re: [PATCH] cvsimport: skip commits that are too recent

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <git@...>, <junkio@...>
Date: Monday, January 8, 2007 - 3:17 am

On 1/8/07, Martin Langhoff <martin@catalyst.net.nz> wrote:

Here is the repost with appropriate doco and an override ;-)

In related news, I am trying to debug an import that consistently
skips over remote commits... which is bad, bad news. The culprit seems
to be cvsps -- it skips commits it clearly knows about, and I'm not
sure why. I do think those were commits that cvsps saw half-baked in
the first place.

Passing -x to cvsps does bring those commits back, cvsps with -x can
afford to rewrite history a little bit. As long as the history being
rewritten is not too old we are safe. So with this patch, passing -x
is safer, assuming that 10 minutes is enough of a time window for
cvsps to change opinion about the project history.

 (Before you ask: from a data correctness, this is a fine mess.)

For this repo, I'll start running cvsimport with -o ' -x ' and see how
it behaves. Time-wise, the bandwidth usage and cpu times are roughly
similar for me using --cvs-direct. The patch to do it by default in
cvsimport is trivial, but I'm not entirely happy with the concept just
now.

In any case -- this should be a bit of a warning. cvsps is not
particularly reliable (not that cvs data ever is!), and passing -o '
-x' may help.

cheers,


martin
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] cvsimport: skip commits that are too recent, Martin Langhoff, (Mon Jan 8, 2:43 am)
Re: [PATCH] cvsimport: skip commits that are too recent, Robin Rosenberg, (Thu Jan 11, 4:22 am)
Re: [PATCH] cvsimport: skip commits that are too recent, Martin Langhoff, (Thu Jan 11, 4:18 pm)
Re: [PATCH] cvsimport: skip commits that are too recent, Martin Langhoff, (Mon Jan 8, 3:17 am)
Re: [PATCH] cvsimport: skip commits that are too recent, Martin Langhoff, (Mon Jan 8, 4:24 am)
speck-geostationary