Re: git-cvsimport-3 and incremental imports

2013-01-21 Thread John Keeping
On Sun, Jan 20, 2013 at 06:20:08PM -0500, Eric S. Raymond wrote: John Keeping j...@keeping.me.uk: I don't think there is any way to solve this without giving cvsps more information, probably the last commit time for all git branches, but perhaps I'm missing a fast-import feature that can help

Re: git-cvsimport-3 and incremental imports

2013-01-21 Thread Eric S. Raymond
John Keeping j...@keeping.me.uk: But this is nothing more than a sticking plaster that happens to do enough in this particular case I'm beginning to think that's the best outcome we ever get in this problem domain... - if the Git repository happened to be on a different

Re: git-cvsimport-3 and incremental imports

2013-01-21 Thread John Keeping
On Mon, Jan 21, 2013 at 06:28:53AM -0500, Eric S. Raymond wrote: John Keeping j...@keeping.me.uk: But this is nothing more than a sticking plaster that happens to do enough in this particular case I'm beginning to think that's the best outcome we ever get in this problem domain... I don't

Re: git-cvsimport-3 and incremental imports

2013-01-21 Thread Eric S. Raymond
John Keeping j...@keeping.me.uk: I also disagree that cvsps outputs commits *newer* than T since it will also output commits *at* T, which is what I changed with the patch in my previous message. Ah. OK, that is yet another bug inherited from 2.x - the code doesn't match the documented (and

Re: git-cvsimport-3 and incremental imports

2013-01-21 Thread John Keeping
On Mon, Jan 21, 2013 at 07:43:40AM -0500, Eric S. Raymond wrote: John Keeping j...@keeping.me.uk: I also disagree that cvsps outputs commits *newer* than T since it will also output commits *at* T, which is what I changed with the patch in my previous message. Ah. OK, that is yet another

Re: git-cvsimport-3 and incremental imports

2013-01-21 Thread Eric S. Raymond
John Keeping j...@keeping.me.uk: Ah. OK, that is yet another bug inherited from 2.x - the code doesn't match the documented (and correct) behavior. Please send me a patch against the cvsps repo, I'll merge it. Should now be in your inbox. Received, merged, tested, and cvsps-3.10 has

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Eric S. Raymond
John Keeping j...@keeping.me.uk: I don't think there is any way to solve this without giving cvsps more information, probably the last commit time for all git branches, but perhaps I'm missing a fast-import feature that can help solve this problem. Yes, you are. The magic incantation is

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Jonathan Nieder
Eric S. Raymond wrote: You get to integrate this. I think the transition strategy Junio has chosen is seriously mistaken, leading to unnecessary grief for users who will be fooled into thinking it's OK to still use cvsps-2.x. So our choices are: a. support current users, offend ESR, don't

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Jonathan Nieder
Eric S. Raymond wrote: You get to integrate this. I think the transition strategy Junio has chosen is seriously mistaken, leading to unnecessary grief for users who will be fooled into thinking it's OK to still use cvsps-2. Ah, I missed a detail on first reading. I think there has been a

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Junio C Hamano
Jonathan Nieder jrnie...@gmail.com writes: Eric S. Raymond wrote: You get to integrate this. I think the transition strategy Junio has chosen is seriously mistaken, leading to unnecessary grief for users who will be fooled into thinking it's OK to still use cvsps-2.x. So our choices are:

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: ..., so I do not think it is a big loss to rely on the same assumptions and choose b. from our point of view. Of course the last sentence is a typo: choose c. is what I meant. -- To unsubscribe from this list: send the line unsubscribe git in the body

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Eric S. Raymond
Jonathan Nieder jrnie...@gmail.com: Junio proposed a transition strategy, but I don't think it's fair to say he has chosen it without discussion or is imposing it on you. I have said everything I could about the bad effects of encouraging people to continue to use cvsps-2.x, it didn't budge

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Jonathan Nieder
Eric S. Raymond wrote: I have said everything I could about the bad effects of encouraging people to continue to use cvsps-2.x, it didn't budge Junio an inch, and I'm tired of fighting about it. What I think you misunderstood is that Junio is not the person you would have needed to convince.

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Junio C Hamano
Eric S. Raymond e...@thyrsus.com writes: Jonathan Nieder jrnie...@gmail.com: Junio proposed a transition strategy, but I don't think it's fair to say he has chosen it without discussion or is imposing it on you. I have said everything I could about the bad effects of encouraging people to

Re: git-cvsimport-3 and incremental imports

2013-01-20 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: If you want to abandon cvsps2 users, that is perfectly fine by me. As long as cvsps3 and cvsimport-3 combo works, Git before this series will have a _working_ cvsimport as far as I am concerned. The above obviously is Git _after_ this series. Git