Stefan Beller <sbeller@xxxxxxxxxx> writes: > + if (transport->smart_options > + && transport->smart_options->transport_version) { > + buf = xmalloc(strlen(remote_program) + 12); > + sprintf(buf, "%s-%d", remote_program, > + transport->smart_options->transport_version); > + remote_program = buf; > + } Bikeshedding: so the versioning scheme is that the current one is zero, and the next one is two? I would have expected that there would be something like if (...->version < 2) { ... append "-%d" ... } involved. -- 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