Re: [PATCH v5 00/15] config: make git_config_set die on failure

> [...]
> Junio pointed out that it might not always be the most sensible
> thing to die when install_branch_config fails. After thinking
> about it I changed the behavior of the function to print an error
> and advise message and return an error code. The error code is
> then only used by the `git branch --set-upstream-to=` command to
> abort early, as its main intent will usually be to set the
> tracking information. The other callers (related to git-clone and
> git-push) simply ignore the returned value while the messages are
> still printed.
>
> I think it does make sense to not abort clones and pushes when
> the function fails. Setting the upstream information is only a
> small part of these commands and especially when cloning a large
> repository it is harmful to die as this would delete everything
> that has just been cloned. The user can still fix up the remote
> tracking branch afterwards.
>
> Interdiff between v4 and v5:
>
> diff --git a/branch.c b/branch.c
> @@ -49,7 +49,13 @@ static int should_setup_rebase(const char *origin)
> +static const char tracking_advice[] =
> +N_("\n"
> +"After fixing the error cause you may try to fix up\n"
> +"the remote tracking information by invoking\n"
> +"\"git branch --set-upstream-to=\".");