On 12-07-05 06:52 PM, Junio C Hamano wrote: > marcnarc@xxxxxxxxxxx writes: > >> From: Marc Branchaud <marcnarc@xxxxxxxxxxx> >> >> Signed-off-by: Marc Branchaud <marcnarc@xxxxxxxxxxx> >> --- >> builtin/clone.c | 2 ++ >> t/t5601-clone.sh | 10 ++++++++++ >> t/t5702-clone-options.sh | 7 +++++-- >> 3 files changed, 17 insertions(+), 2 deletions(-) >> >> diff --git a/builtin/clone.c b/builtin/clone.c >> index a4d8d25..b198456 100644 >> --- a/builtin/clone.c >> +++ b/builtin/clone.c >> @@ -770,6 +770,8 @@ int cmd_clone(int argc, const char **argv, const char *prefix) >> git_config_set(key.buf, repo); >> strbuf_reset(&key); >> >> + git_config_set("remote.default", option_origin); >> + > > Is this something we would want to do unconditionally? If so why? I think so, yes. > Or is this what we want to do only when the "--origin name" option > is used? If remote.default isn't set, then if someone does git remote rename origin foo the default remote will still be "origin" (modulo the currently-checked-out branch stuff). M. -- 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