{"thread":{"id":"24204","subject":"[RFC PATCH 1/2] New remote helper git-remote-fd","startedAt":"2010-06-26T14:20:16Z","lastAt":"2010-06-26T14:33:05Z","messageCount":3,"participants":["Ilari Liusvaara","Sverre Rabbelier"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"144315","messageId":"1277562017-3316-1-git-send-email-ilari.liusvaara@elisanet.fi","threadId":"24204","inReplyTo":null,"subject":"[RFC PATCH 1/2] New remote helper git-remote-fd","fromName":"Ilari Liusvaara","fromEmail":"ilari.liusvaara@elisanet.fi","sentAt":"2010-06-26T14:20:16Z","receivedAt":"2010-06-26T14:20:16Z","isPatch":true,"sender":{"key":"ilari.liusvaara@elisanet.fi","avatar":null},"body":"This remote helper reflects the outgoing smart transport stream back to\ncaller. This is useful if some frontend wants to handle the transport\nstream for some reason (e.g. offering more options for SSH).\n\nSigned-off-by: Ilari Liusvaara <ilari.liusvaara@elisanet.fi>\n---\n Makefile            |    1 +\n builtin.h           |    1 +\n builtin/remote-fd.c |   88 ++++++++++++++++++++++++++++++++++\n git.c               |    1 +\n transport-helper.c  |  130 +++++++++++++++++++++++++++++++++++++++++++++++++++\n transport.h         |    1 +\n 6 files changed, 222 insertions(+), 0 deletions(-)\n create mode 100644 builtin/remote-fd.c\n\ndiff --git a/Makefile b/Makefile\nindex 5fa893c..ad53b52 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -702,6 +702,7 @@ BUILTIN_OBJS += builtin/read-tree.o\n BUILTIN_OBJS += builtin/receive-pack.o\n BUILTIN_OBJS += builtin/reflog.o\n BUILTIN_OBJS += builtin/remote.o\n+BUILTIN_OBJS += builtin/remote-fd.o\n BUILTIN_OBJS += builtin/replace.o\n BUILTIN_OBJS += builtin/rerere.o\n BUILTIN_OBJS += builtin/reset.o\ndiff --git a/builtin.h b/builtin.h\nindex 5c887ef..af60b28 100644\n--- a/builtin.h\n+++ b/builtin.h\n@@ -138,5 +138,6 @@ extern int cmd_verify_pack(int argc, const char **argv, const char *prefix);\n extern int cmd_show_ref(int argc, const char **argv, const char *prefix);\n extern int cmd_pack_refs(int argc, const char **argv, const char *prefix);\n extern int cmd_replace(int argc, const char **argv, const char *prefix);\n+extern int cmd_remote_fd(int argc, const char **argv, const char *prefix);\n \n #endif\ndiff --git a/builtin/remote-fd.c b/builtin/remote-fd.c\nnew file mode 100644\nindex 0000000..08ff522\n--- /dev/null\n+++ b/builtin/remote-fd.c\n@@ -0,0 +1,88 @@\n+#include \"git-compat-util.h\"\n+#include \"transport.h\"\n+#include <errno.h>\n+#include <stdlib.h>\n+#include <string.h>\n+#include <stdio.h>\n+#include <unistd.h>\n+\n+\n+/*\n+ * URL syntax:\n+ *\t'fd::<inoutfd>[/<anything>]'\t\tRead/write socket pair\n+ *\t\t\t\t\t\t<inoutfd>.\n+ *\t'fd::<infd>,<outfd>[/<anything>]'\tRead pipe <infd> and write\n+ *\t\t\t\t\t\tpipe <outfd>.\n+ *\t[foo] indicates 'foo' is optional. <anything> is any string.\n+ *\n+ * The data output to <outfd>/<inoutfd> should be passed unmolested to\n+ * git-receive-pack/git-upload-pack/git-upload-archive and output of\n+ * git-receive-pack/git-upload-pack/git-upload-archive should be passed\n+ * unmolested to <infd>/<inoutfd>.\n+ *\n+ */\n+\n+int input_fd = -1;\n+int output_fd = -1;\n+\n+#define MAXCOMMAND 4096\n+\n+static int command_loop()\n+{\n+\tchar buffer[MAXCOMMAND];\n+\n+\twhile (1) {\n+\t\tif (!fgets(buffer, MAXCOMMAND - 1, stdin))\n+\t\t\texit(0);\n+\t\t//Strip end of line characters.\n+\t\twhile (isspace((unsigned char)buffer[strlen(buffer) - 1]))\n+\t\t\tbuffer[strlen(buffer) - 1] = 0;\n+\n+\t\tif (!strcmp(buffer, \"capabilities\")) {\n+\t\t\tprintf(\"*connect\\n\\n\");\n+\t\t\tfflush(stdout);\n+\t\t} else if (!strncmp(buffer, \"connect \", 8)) {\n+\t\t\tprintf(\"\\n\");\n+\t\t\tfflush(stdout);\n+\t\t\treturn bidirectional_transfer_loop(input_fd,\n+\t\t\t\toutput_fd);\n+\t\t} else {\n+\t\t\tfprintf(stderr, \"Bad command\");\n+\t\t\treturn 1;\n+\t\t}\n+\t}\n+}\n+\n+int cmd_remote_fd(int argc, const char **argv, const char *prefix)\n+{\n+\tchar* end;\n+\tunsigned long r;\n+\n+\tif (argc < 3) {\n+\t\tfprintf(stderr, \"Error: URL missing\");\n+\t\texit(1);\n+\t}\n+\n+\tr = strtoul(argv[2], &end, 10);\n+\tinput_fd = (int)r;\n+\n+\tif ((*end != ',' && *end !='/' && *end) || end == argv[2]) {\n+\t\tfprintf(stderr, \"Error: Bad URL syntax\");\n+\t\texit(1);\n+\t}\n+\n+\tif (*end == '/' || !*end) {\n+\t\toutput_fd = input_fd;\n+\t} else {\n+\t\tchar* end2;\n+\t\tr = strtoul(end + 1, &end2, 10);\n+\t\toutput_fd = (int)r;\n+\n+\t\tif ((*end2 !='/' && *end2) || end2 == end + 1) {\n+\t\t\tfprintf(stderr, \"Error: Bad URL syntax\");\n+\t\t\texit(1);\n+\t\t}\n+\t}\n+\n+\treturn command_loop();\n+}\ndiff --git a/git.c b/git.c\nindex 99f0363..91de3d6 100644\n--- a/git.c\n+++ b/git.c\n@@ -368,6 +368,7 @@ static void handle_internal_command(int argc, const char **argv)\n \t\t{ \"receive-pack\", cmd_receive_pack },\n \t\t{ \"reflog\", cmd_reflog, RUN_SETUP },\n \t\t{ \"remote\", cmd_remote, RUN_SETUP },\n+\t\t{ \"remote-fd\", cmd_remote_fd, 0 },\n \t\t{ \"replace\", cmd_replace, RUN_SETUP },\n \t\t{ \"repo-config\", cmd_config },\n \t\t{ \"rerere\", cmd_rerere, RUN_SETUP },\ndiff --git a/transport-helper.c b/transport-helper.c\nindex 0381de5..e07aa40 100644\n--- a/transport-helper.c\n+++ b/transport-helper.c\n@@ -862,3 +862,133 @@ int transport_helper_init(struct transport *transport, const char *name)\n \ttransport->smart_options = &(data->transport_options);\n \treturn 0;\n }\n+\n+\n+#define BUFFERSIZE 4096\n+\n+/* Copy data from stdin to output and from input to stdout. */\n+int bidirectional_transfer_loop(int input, int output)\n+{\n+\tstruct pollfd polls[4];\n+\tchar in_buffer[BUFFERSIZE];\n+\tchar out_buffer[BUFFERSIZE];\n+\tsize_t in_buffer_use = 0;\n+\tsize_t out_buffer_use = 0;\n+\tint in_hup = 0;\n+\tint out_hup = 0;\n+\tint socket_mode = 0;\n+\tint input_index = 2;\n+\tint output_index = 3;\n+\tint poll_count = 4;\n+\n+\tif(input == output) {\n+\t\toutput_index = input_index;\n+\t\tpoll_count = 3;\n+\t\tsocket_mode = 1;\n+\t}\n+\n+\twhile(in_buffer_use || out_buffer_use || !in_hup || !out_hup) {\n+\t\tint r, i;\n+\t\t/* Set up the poll and do it. */\n+\t\tpolls[0].fd = 0;\n+\t\tpolls[1].fd = 1;\n+\t\tpolls[input_index].fd = input;\n+\t\tpolls[output_index].fd = output;\n+\t\tfor(i = 0; i < 4; i++)\n+\t\t\tpolls[i].events = polls[i].revents = 0;\n+\n+\t\tif(in_buffer_use > 0)\n+\t\t\tpolls[output_index].events |= POLLOUT;\n+\t\tif(in_buffer_use < BUFFERSIZE && !in_hup)\n+\t\t\tpolls[0].events |= POLLIN;\n+\t\tif(out_buffer_use > 0)\n+\t\t\tpolls[1].events |= POLLOUT;\n+\t\tif(out_buffer_use < BUFFERSIZE && !out_hup)\n+\t\t\tpolls[input_index].events |= POLLIN;\n+\t\tr = poll(polls, poll_count, -1);\n+\t\tif(r < 0) {\n+\t\t\tif(errno == EWOULDBLOCK || errno == EAGAIN ||\n+\t\t\t\terrno == EINTR)\n+\t\t\t\tcontinue;\n+\t\t\tperror(\"poll failed\");\n+\t\t\treturn 1;\n+\t\t} else if(r == 0)\n+\t\t\tcontinue;\n+\n+\t\t/* Something interesting has happened... */\n+\t\tif(polls[0].revents & (POLLIN | POLLHUP)) {\n+\t\t\t/* Stdin is readable. */\n+\t\t\tr = read(0, in_buffer + in_buffer_use, BUFFERSIZE -\n+\t\t\t\tin_buffer_use);\n+\t\t\tif(r < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&\n+\t\t\t\terrno != EINTR) {\n+\t\t\t\tperror(\"read(git) failed\");\n+\t\t\t\treturn 1;\n+\t\t\t} else if(r == 0) {\n+\t\t\t\tin_hup = 1;\n+\t\t\t\tif(!in_buffer_use) {\n+\t\t\t\t\tif(socket_mode)\n+\t\t\t\t\t\tshutdown(output, SHUT_WR);\n+\t\t\t\t\telse\n+\t\t\t\t\t\tclose(output);\n+\t\t\t\t}\n+\t\t\t} else\n+\t\t\t\tin_buffer_use += r;\n+\t\t}\n+\n+\t\tif(polls[input_index].revents & (POLLIN | POLLHUP)) {\n+\t\t\t/* Connection is readable. */\n+\t\t\tr = read(input, out_buffer + out_buffer_use,\n+\t\t\t\tBUFFERSIZE - out_buffer_use);\n+\t\t\tif(r < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&\n+\t\t\t\terrno != EINTR) {\n+\t\t\t\tperror(\"read(connection) failed\");\n+\t\t\t\treturn 1;\n+\t\t\t} else if(r == 0) {\n+\t\t\t\tout_hup = 1;\n+\t\t\t\tif(!out_buffer_use)\n+\t\t\t\t\tclose(1);\n+\t\t\t} else\n+\t\t\t\tout_buffer_use += r;\n+\t\t}\n+\n+\t\tif(polls[1].revents & POLLOUT) {\n+\t\t\t/* Stdout is writable. */\n+\t\t\tr = write(1, out_buffer, out_buffer_use);\n+\t\t\tif(r < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&\n+\t\t\t\terrno != EINTR) {\n+\t\t\t\tperror(\"write(git) failed\");\n+\t\t\t\treturn 1;\n+\t\t\t} else {\n+\t\t\t\tout_buffer_use -= r;\n+\t\t\t\tif(out_buffer_use > 0)\n+\t\t\t\t\tmemmove(out_buffer, out_buffer + r,\n+\t\t\t\t\t\tout_buffer_use);\n+\t\t\t\tif(out_hup && !out_buffer_use)\n+\t\t\t\t\tclose(1);\n+\t\t\t}\n+\t\t}\n+\n+\t\tif(polls[output_index].revents & POLLOUT) {\n+\t\t\t/* Connection is writable. */\n+\t\t\tr = write(output, in_buffer, in_buffer_use);\n+\t\t\tif(r < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&\n+\t\t\t\terrno != EINTR) {\n+\t\t\t\tperror(\"write(connection) failed\");\n+\t\t\t\treturn 1;\n+\t\t\t} else {\n+\t\t\t\tin_buffer_use -= r;\n+\t\t\t\tif(in_buffer_use > 0)\n+\t\t\t\t\tmemmove(in_buffer, in_buffer + r,\n+\t\t\t\t\t\tin_buffer_use);\n+\t\t\t\tif(in_hup && !in_buffer_use) {\n+\t\t\t\t\tif(socket_mode)\n+\t\t\t\t\t\tshutdown(output, SHUT_WR);\n+\t\t\t\t\telse\n+\t\t\t\t\t\tclose(output);\n+\t\t\t\t}\n+\t\t\t}\n+\t\t}\n+\t}\n+\treturn 0;\n+}\ndiff --git a/transport.h b/transport.h\nindex c59d973..e803c0e 100644\n--- a/transport.h\n+++ b/transport.h\n@@ -154,6 +154,7 @@ int transport_connect(struct transport *transport, const char *name,\n \n /* Transport methods defined outside transport.c */\n int transport_helper_init(struct transport *transport, const char *name);\n+int bidirectional_transfer_loop(int input, int output);\n \n /* common methods used by transport.c and builtin-send-pack.c */\n void transport_verify_remote_names(int nr_heads, const char **heads);\n-- \n1.7.1\n"},{"id":"144316","messageId":"1277562017-3316-2-git-send-email-ilari.liusvaara@elisanet.fi","threadId":"24204","inReplyTo":"1277562017-3316-1-git-send-email-ilari.liusvaara@elisanet.fi","subject":"[RFC PATCH 2/2] New remote helper: git-remote-ext","fromName":"Ilari Liusvaara","fromEmail":"ilari.liusvaara@elisanet.fi","sentAt":"2010-06-26T14:20:17Z","receivedAt":"2010-06-26T14:20:17Z","isPatch":true,"sender":{"key":"ilari.liusvaara@elisanet.fi","avatar":null},"body":"Invokes specified command and directs smart transport streams to its\nstdin/stdout. Handy for e.g. invoking ssh with some one-off parameters.\n\nSigned-off-by: Ilari Liusvaara <ilari.liusvaara@elisanet.fi>\n---\nExample URL: \"ext::ssh -i foo.pub user@host.example \\\\S 'some/repo'\".\n\n Makefile             |    1 +\n builtin.h            |    1 +\n builtin/remote-ext.c |  195 ++++++++++++++++++++++++++++++++++++++++++++++++++\n git.c                |    1 +\n 4 files changed, 198 insertions(+), 0 deletions(-)\n create mode 100644 builtin/remote-ext.c\n\ndiff --git a/Makefile b/Makefile\nindex ad53b52..88e752f 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -702,6 +702,7 @@ BUILTIN_OBJS += builtin/read-tree.o\n BUILTIN_OBJS += builtin/receive-pack.o\n BUILTIN_OBJS += builtin/reflog.o\n BUILTIN_OBJS += builtin/remote.o\n+BUILTIN_OBJS += builtin/remote-ext.o\n BUILTIN_OBJS += builtin/remote-fd.o\n BUILTIN_OBJS += builtin/replace.o\n BUILTIN_OBJS += builtin/rerere.o\ndiff --git a/builtin.h b/builtin.h\nindex af60b28..eb9074d 100644\n--- a/builtin.h\n+++ b/builtin.h\n@@ -139,5 +139,6 @@ extern int cmd_show_ref(int argc, const char **argv, const char *prefix);\n extern int cmd_pack_refs(int argc, const char **argv, const char *prefix);\n extern int cmd_replace(int argc, const char **argv, const char *prefix);\n extern int cmd_remote_fd(int argc, const char **argv, const char *prefix);\n+extern int cmd_remote_ext(int argc, const char **argv, const char *prefix);\n \n #endif\ndiff --git a/builtin/remote-ext.c b/builtin/remote-ext.c\nnew file mode 100644\nindex 0000000..e6caca4\n--- /dev/null\n+++ b/builtin/remote-ext.c\n@@ -0,0 +1,195 @@\n+#include \"git-compat-util.h\"\n+#include \"transport.h\"\n+#include \"run-command.h\"\n+#include <errno.h>\n+#include <stdlib.h>\n+#include <string.h>\n+#include <stdio.h>\n+#include <unistd.h>\n+\n+/*\n+ * URL syntax:\n+ *\t'command [arg1 [arg2 [...]]]'\tInvoke command with given arguments.\n+ *\tSpecial characters:\n+ *\t'\\ ': Literal space in argument.\n+ *\t'\\\\': Literal backslash.\n+ *\t'\\S': Name of service (git-upload-pack/git-upload-archive/\n+ *\t\tgit-receive-pack.\n+ *\t'\\s': Same as \\s, but with possible git- prefix stripped.\n+ */\n+\n+static char *strip_escapes(const char *str, const char *service,\n+\tconst char **next)\n+{\n+\tchar *ret;\n+\tsize_t rpos = 0;\n+\tsize_t wpos = 0;\n+\tsize_t finallen = 0;\n+\tint escape = 0;\n+\tsize_t pslen = 0;\n+\tsize_t pSlen = 0;\n+\tsize_t psoff = 0;\n+\n+\t/* Calculate prefix length for \\s and lengths for \\s and \\S */\n+\tif(!strncmp(service, \"git-\", 4))\n+\t\tpsoff = 4;\n+\tpSlen = strlen(service);\n+\tpslen = pSlen - psoff;\n+\n+\t/* Calculate output length and start of next argument. */\n+\twhile (str[rpos] && (escape || str[rpos] != ' ')) {\n+\t\tif (escape) {\n+\t\t\tswitch (str[rpos]) {\n+\t\t\tcase ' ':\n+\t\t\tcase '\\\\':\n+\t\t\t\tfinallen++;\n+\t\t\t\tbreak;\n+\t\t\tcase 's':\n+\t\t\t\tfinallen += pslen;\n+\t\t\t\tbreak;\n+\t\t\tcase 'S':\n+\t\t\t\tfinallen += pSlen;\n+\t\t\t\tbreak;\n+\t\t\tdefault:\n+\t\t\t\tdie(\"Bad remote-ext placeholder '\\\\%c'.\",\n+\t\t\t\t\tstr[rpos]);\n+\t\t\t}\n+\t\t\tescape = 0;\n+\t\t} else\n+\t\t\tswitch (str[rpos]) {\n+\t\t\tcase '\\\\':\n+\t\t\t\tescape = 1;\n+\t\t\t\tbreak;\n+\t\t\tdefault:\n+\t\t\t\tfinallen++;\n+\t\t\t\tbreak;\n+\t\t\t}\n+\t\trpos++;\n+\t}\n+\tif (escape && !str[rpos])\n+\t\tdie(\"remote-ext command has incomplete placeholder\");\n+\t*next = str + rpos;\n+\tif (**next == ' ')\n+\t\t++*next;\t/* Skip over space */\n+\n+\t/*\n+\t * Do the actual placeholder substitution. The string will be short\n+\t * enough not to overflow integers.\n+\t */\n+\tret = xmalloc(finallen + 1);\n+\trpos = 0;\n+\tescape = 0;\n+\twhile (str[rpos] && (escape || str[rpos] != ' ')) {\n+\t\tif (escape) {\n+\t\t\tswitch(str[rpos]) {\n+\t\t\tcase ' ':\n+\t\t\tcase '\\\\':\n+\t\t\t\tret[wpos++] = str[rpos];\n+\t\t\t\tbreak;\n+\t\t\tcase 's':\n+\t\t\t\tstrcpy(ret + wpos, service + psoff);\n+\t\t\t\twpos += pslen;\n+\t\t\t\tbreak;\n+\t\t\tcase 'S':\n+\t\t\t\tstrcpy(ret + wpos, service);\n+\t\t\t\twpos += pSlen;\n+\t\t\t\tbreak;\n+\t\t\t}\n+\t\t\tescape = 0;\n+\t\t} else\n+\t\t\tswitch(str[rpos]) {\n+\t\t\tcase '\\\\':\n+\t\t\t\tescape = 1;\n+\t\t\t\tbreak;\n+\t\t\tdefault:\n+\t\t\t\tret[wpos++] = str[rpos];\n+\t\t\t\tbreak;\n+\t\t\t}\n+\t\trpos++;\n+\t}\n+\tret[wpos] = 0;\n+\treturn ret;\n+}\n+\n+/* Should be enough... */\n+#define MAXARGUMENTS 256\n+\n+static const char **parse_argv(const char *arg, const char *service)\n+{\n+\tint arguments = 0;\n+\tint i;\n+\tchar** ret;\n+\tchar *(temparray[MAXARGUMENTS + 1]);\n+\n+\twhile (*arg) {\n+\t\tif(arguments == MAXARGUMENTS)\n+\t\t\tdie(\"remote-ext command has too many arguments\");\n+\t\ttemparray[arguments++] = strip_escapes(arg, service, &arg);\n+\t}\n+\n+\tret = xcalloc(arguments + 1, sizeof(char*));\n+\tfor (i = 0; i < arguments; i++)\n+\t\tret[i] = temparray[i];\n+\n+\treturn (const char**)ret;\n+}\n+\n+static int run_child(const char *arg, const char *service)\n+{\n+\tint r;\n+\tstruct child_process child;\n+\n+\tmemset(&child, 0, sizeof(child));\n+\tchild.in = -1;\n+\tchild.out = -1;\n+\tchild.err = 0;\n+\tchild.argv = parse_argv(arg, service);\n+\n+\tif (start_command(&child) < 0) {\n+\t\tfprintf(stderr, \"fatal: Can't run specified command\");\n+\t\treturn 128;\n+\t}\n+\tr = bidirectional_transfer_loop(child.out, child.in);\n+\tif (!r)\n+\t\tr = finish_command(&child);\n+\telse\n+\t\tfinish_command(&child);\n+\treturn r;\n+}\n+\n+#define MAXCOMMAND 4096\n+\n+static int command_loop(const char *child)\n+{\n+\tchar buffer[MAXCOMMAND];\n+\n+\twhile (1) {\n+\t\tif (!fgets(buffer, MAXCOMMAND - 1, stdin))\n+\t\t\texit(0);\n+\t\t//Strip end of line characters.\n+\t\twhile (isspace((unsigned char)buffer[strlen(buffer) - 1]))\n+\t\t\tbuffer[strlen(buffer) - 1] = 0;\n+\n+\t\tif (!strcmp(buffer, \"capabilities\")) {\n+\t\t\tprintf(\"*connect\\n\\n\");\n+\t\t\tfflush(stdout);\n+\t\t} else if (!strncmp(buffer, \"connect \", 8)) {\n+\t\t\tprintf(\"\\n\");\n+\t\t\tfflush(stdout);\n+\t\t\treturn run_child(child, buffer + 8);\n+\t\t} else {\n+\t\t\tfprintf(stderr, \"Bad command\");\n+\t\t\treturn 1;\n+\t\t}\n+\t}\n+}\n+\n+int cmd_remote_ext(int argc, const char **argv, const char *prefix)\n+{\n+\tif (argc < 3) {\n+\t\tfprintf(stderr, \"Error: URL missing\");\n+\t\texit(1);\n+\t}\n+\n+\treturn command_loop(argv[2]);\n+}\ndiff --git a/git.c b/git.c\nindex 91de3d6..2cc0826 100644\n--- a/git.c\n+++ b/git.c\n@@ -368,6 +368,7 @@ static void handle_internal_command(int argc, const char **argv)\n \t\t{ \"receive-pack\", cmd_receive_pack },\n \t\t{ \"reflog\", cmd_reflog, RUN_SETUP },\n \t\t{ \"remote\", cmd_remote, RUN_SETUP },\n+\t\t{ \"remote-ext\", cmd_remote_ext, 0 },\n \t\t{ \"remote-fd\", cmd_remote_fd, 0 },\n \t\t{ \"replace\", cmd_replace, RUN_SETUP },\n \t\t{ \"repo-config\", cmd_config },\n-- \n1.7.1\n"},{"id":"144317","messageId":"AANLkTinyn5gLa1UHsFVCPy2yTsP5O16yEq2uzelGszGt@mail.gmail.com","threadId":"24204","inReplyTo":"1277562017-3316-1-git-send-email-ilari.liusvaara@elisanet.fi","subject":"Re: [RFC PATCH 1/2] New remote helper git-remote-fd","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2010-06-26T14:33:05Z","receivedAt":"2010-06-26T14:33:05Z","isPatch":true,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Sat, Jun 26, 2010 at 16:20, Ilari Liusvaara\n<ilari.liusvaara@elisanet.fi> wrote:\n> This remote helper reflects the outgoing smart transport stream back to\n> caller. This is useful if some frontend wants to handle the transport\n> stream for some reason (e.g. offering more options for SSH).\n\nI don't understand the intended use of either this helper, nor the one\nin the next patch. Can you give a more concrete explanation?\n\n-- \nCheers,\n\nSverre Rabbelier\n"}]}