On Wed, Jul 20, 2016 at 6:00 PM, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote: >> - needs_cloning, ce->name); >> + needs_cloning, sub->name, sub->path); > > Are there any restrictions on what characters a submodule name can > contain? Does this need e.g. to be quoted with sq_quote? Oh good point, this ought to fail with weird names. So instead of quoting it, let's just drop this patch. > > Thanks, > Jonathan -- 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