From: Junio C Hamano <gitster@pobox.com>
To: John Keeping <john@keeping.me.uk>
Cc: git@vger.kernel.org, "Eric S. Raymond" <esr@thyrsus.com>,
Chris Rorvick <chris@rorvick.com>
Subject: Re: [PATCH] git-cvsimport.txt: cvsps-2 is deprecated
Date: Thu, 24 Jan 2013 12:58:02 -0800 [thread overview]
Message-ID: <7vmwvy9u3p.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: <20130124201330.GT7498@serenity.lan> (John Keeping's message of "Thu, 24 Jan 2013 20:13:30 +0000")
John Keeping <john@keeping.me.uk> writes:
> On Thu, Jan 24, 2013 at 12:04:11PM -0800, Junio C Hamano wrote:
>> John Keeping <john@keeping.me.uk> writes:
>>
>> > git-cvsimport relies on version 2 of cvsps and does not work with the
>> > new version 3. Since cvsps 3.x does not currently work as well as
>> > version 2 for incremental import, document this fact.
>> >
>> > Specifically, there is no way to make new git-cvsimport that supports
>> > cvsps 3.x and have a seamless transition for existing users since cvsps
>> > 3.x needs a time from which to continue importing and git-cvsimport does
>> > not save the time of the last import or import into a specific namespace
>> > so there is no safe way to calculate the time of the last import.
>>
>> Isn't the whole "and git-cvsimport does not save the time..." part
>> something that can be fixed in the new cvsimport that reads the
>> output from cvsps3?
>
> Yes it can be fixed there (and I have patches to do that) - my argument
> here is that there cannot be a seamless upgrade for people who are
> currently using git-cvsimport incrementally. If you don't have that
> file then how do you create it to reflect the current state of your
> repository?
I am not disagreeing with your patch text to warn the current users
that cvsimport will be made unusable if they lose cvsps2, and they
are better off using other tools when doing a whole-history import.
I was trying to make sure that my understanding of the current
situation and future possibilities matches yours.
Thanks, I think you clarified the situation better with your
response.
next prev parent reply other threads:[~2013-01-24 20:58 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-22 22:44 What's cooking in git.git (Jan 2013, #08; Tue, 22) Junio C Hamano
2013-01-22 23:45 ` John Keeping
2013-01-23 0:11 ` Junio C Hamano
2013-01-23 9:28 ` John Keeping
2013-01-23 13:26 ` Chris Rorvick
2013-01-23 13:55 ` John Keeping
2013-01-23 17:13 ` Junio C Hamano
2013-01-23 21:12 ` John Keeping
2013-01-24 5:04 ` Junio C Hamano
2013-01-24 19:18 ` [PATCH] git-cvsimport.txt: cvsps-2 is deprecated John Keeping
2013-01-24 20:04 ` Junio C Hamano
2013-01-24 20:13 ` John Keeping
2013-01-24 20:58 ` Junio C Hamano [this message]
2013-01-25 4:55 ` What's cooking in git.git (Jan 2013, #08; Tue, 22) Chris Rorvick
2013-01-25 9:09 ` John Keeping
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=7vmwvy9u3p.fsf@alter.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=chris@rorvick.com \
--cc=esr@thyrsus.com \
--cc=git@vger.kernel.org \
--cc=john@keeping.me.uk \
/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).