David Aguilar <dav...@gmail.com> writes:

> Teach resolve_deleted_merge() to honor the mergetool.keepBackup and
> mergetool.keepTemporaries configuration knobs.
>
> This ensures that the worktree is kept pristine when resolving deletion
> conflicts with the variables both set to false.
>
> Signed-off-by: David Aguilar <dav...@gmail.com>
> ---
>  git-mergetool.sh | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/git-mergetool.sh b/git-mergetool.sh
> index 9f77e3a..615265d 100755
> --- a/git-mergetool.sh
> +++ b/git-mergetool.sh
> @@ -126,7 +126,12 @@ resolve_deleted_merge () {
>               case "$ans" in
>               [mMcC]*)
>                       git add -- "$MERGED"
> -                     cleanup_temp_files --save-backup
> +                     if "$merge_keep_backup" = "true"

The command run as the "if" condition is probably "test", like in
the other hunk?

> +                     then
> +                             cleanup_temp_files --save-backup
> +                     else
> +                             cleanup_temp_files
> +                     fi
>                       return 0
>                       ;;
>               [dD]*)
> @@ -135,6 +140,10 @@ resolve_deleted_merge () {
>                       return 0
>                       ;;
>               [aA]*)
> +                     if test "$merge_keep_temporaries" = "false"
> +                     then
> +                             cleanup_temp_files
> +                     fi
>                       return 1
>                       ;;
>               esac
--
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