git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Felipe Contreras <felipe.contreras@gmail.com>
To: Junio C Hamano <gitster@pobox.com>, Max Horn <max@quendi.de>
Cc: Felipe Contreras <felipe.contreras@gmail.com>, git@vger.kernel.org
Subject: Re: [PATCH v2 2/3] remote-helpers: move out of contrib
Date: Wed, 23 Apr 2014 16:00:13 -0500	[thread overview]
Message-ID: <535829dd861b1_24448772ece1@nysa.notmuch> (raw)
In-Reply-To: <xmqqa9bbzwc0.fsf@gitster.dls.corp.google.com>

Junio C Hamano wrote:
> Max Horn <max@quendi.de> writes:
> 
> > Perhaps it is OK to move an undocumented remote-helper
> > with known bugs out of contrib.
> 
> We should strive to apply the same criteria as new submission to the
> main part of the system.  And inputs from people like you who have
> more experiences than others on the list in dealing with hg-to-git
> bridging are very much welcomed to identify and document what kind
> of breakages there are, and to come up with the solution to them.

FTR. There are no "known brakages", at least to me. Max Horn tends to hoard
these issues and never report them (unless an opportunity to criticize
git-remote-hg comes, apparently).

The very unlikely issue that nobody has reported about hg multiple heads and gc
I just fixed, and the issue he just reported about 'foo' and 'foo/bar' is newly
reported, and there's no easy way to fix this. My proposal would be to have
some sort of branch mapping mechanism in fast-import, but hardly something that
should prevent the move out of contrib.

-- 
Felipe Contreras

  reply	other threads:[~2014-04-23 21:10 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-21 20:37 [PATCH v2 0/3] remote-helpers: graduate Felipe Contreras
2014-04-21 20:37 ` [PATCH v2 1/3] remote-helpers: squelch python import exceptions Felipe Contreras
2014-04-21 20:37 ` [PATCH v2 2/3] remote-helpers: move out of contrib Felipe Contreras
2014-04-21 21:22   ` Junio C Hamano
2014-04-21 21:24     ` Felipe Contreras
2014-04-21 21:42       ` Junio C Hamano
2014-04-23 13:10   ` Max Horn
2014-04-23 19:20     ` Junio C Hamano
2014-04-23 21:00       ` Felipe Contreras [this message]
2014-04-23 21:30         ` Junio C Hamano
2014-04-23 20:12     ` Junio C Hamano
2014-04-23 20:54     ` Felipe Contreras
2014-04-23 22:41       ` Max Horn
2014-04-24  0:23         ` Felipe Contreras
2014-04-25 22:26           ` Max Horn
2014-04-26  1:28             ` Felipe Contreras
2014-04-21 20:37 ` [PATCH v2 3/3] remote-helpers: move tests " Felipe Contreras

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=535829dd861b1_24448772ece1@nysa.notmuch \
    --to=felipe.contreras@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=max@quendi.de \
    /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).