Thread (11 messages) flat view 11 messages, 4 authors, 2016-06-15

Re: [PATCH 0/4] remote-hg: more improvements

From: Felipe Contreras <hidden>
Date: 2016-06-15 23:01:03

Junio C Hamano wrote:
Felipe Contreras [off-list ref] writes:
quoted
Really? Based on what reasoning? I have proven his reasoning to be
basically wrong.
Perhaps s/proven/convinced myself only/; you didn't prove it to me
and I doubt you proved it to John.
And you are still conveniently avoiding the question:

Based on what reasoning?
quoted
Of course it wasn't a mistake.
I doubt about the "Of course" part.  The first reaction after seeing
that the new "changegroup" is used only inside check_version(3,0)
and nowhere else was to wonder if that import is necessary (or even
safe) for the pre-v3.0 versions.
I don't care about your first reaction. If that was only present in
newer versions, how do you think it would pass the testing on older
versions?

https://travis-ci.org/felipec/git

Normally I would explain the details of why this is the case, and send
the crash regresion fix for v2.0 with a clear explanation, but since you
are adamant in threating git-remote-hg/bzr as just another crappy
contrib script that doesn't even have tests like diff-highlight or
hg-to-git. Why would I care?

The fact that I'm the maintainer and I say it'ss good should be good
enough, and if the current version in "master" renders unusable the
existing Mercurial clones, hey, it's only in contrib, right?

-- 
Felipe Contreras
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help