On Tue, Jun 02, 2015 at 05:02:45PM +0700, Duy Nguyen wrote: > On Tue, Jun 2, 2015 at 7:02 AM, Stefan Beller <sbeller@xxxxxxxxxx> wrote: > > builtin/fetch-pack.c | 22 ++++++++++- > > fetch-pack.c | 109 +++++++++++++++++++++++++++------------------------ > > fetch-pack.h | 1 + > > 3 files changed, 80 insertions(+), 52 deletions(-) > > And the companion changes in transport-helper.c should be in this > patch as well to support smart http. I don't think there is any > problem with how you store the "version" (or "transport_version", you > should probably stick to one name) though. Looking at transport-helper.c, process_connect() looks to need patching, it handles smart transport establishment via remote helpers. Looking at the routine it calls, both the name and exec look to need patching. I think that if process_connect() succeeds, then connect_setup() will hit the "if (data->conn) return 0;" case and exit early. transport-helper.c doesn't look to have anything smart-http specific. -Ilari -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html