Ilari Liusvaara <ilari.liusvaara@xxxxxxxxxxx> wrote: > Signed-off-by: Ilari Liusvaara <ilari.liusvaara@xxxxxxxxxxx> > Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> > Signed-off-by: Ilari Liusvaara <ilari.liusvaara@xxxxxxxxxxx> This is better. > + if (strcmp(name, exec)) { > + r = set_helper_option(transport, "servpath", exec); > + if (r > 0) > + fprintf(stderr, "Warning: Setting remote service path " > + "not supported by protocol.\n"); > + else if (r < 0) > + fprintf(stderr, "Warning: Invalid remote service " > + "path.\n"); Style-nit: We prefer "warning: " with lowercase. -- Shawn. -- 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