Chris Rorvick <ch...@rorvick.com> writes:

>  static void advise_pull_before_push(void)
>  {
>       if (!advice_push_non_ff_current || !advice_push_nonfastforward)
> @@ -241,6 +245,11 @@ static void advise_checkout_pull_push(void)
>       advise(_(message_advice_checkout_pull_push));
>  }
>  
> +static void advise_ref_already_exists(void)
> +{
> +     advise(_(message_advice_ref_already_exists));
> +}
> +
>  static int push_with_options(struct transport *transport, int flags)
>  {
>       int err;
> @@ -272,6 +281,8 @@ static int push_with_options(struct transport *transport, 
> int flags)
>                       advise_use_upstream();
>               else
>                       advise_checkout_pull_push();
> +     } else if (reject_reasons & REJECT_ALREADY_EXISTS) {
> +             advise_ref_already_exists();
>       }

The existing advise_* functions that are called from this function
honor the advice.* configuration, and advise_ref_already_exists()
would want to follow suit here (it is OK to do so as a follow-up
patch without further rerolling the entire series).

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to