# [PATCHv2] Make difftool.prompt fall back to mergetool.prompt 3 messages from 2010-01-22 to 2010-01-22. Participants: Sebastian Schuberth, David Aguilar. Thread: https://gitlist.dev/t/22342 ## Sebastian Schuberth, 2010-01-22 16:36 Subject: [PATCHv2] Make difftool.prompt fall back to mergetool.prompt Message-ID: <4B59D414.4020507@gmail.com> URL: https://gitlist.dev/e/4B59D414.4020507%40gmail.com ``` The documentation states that "git-difftool falls back to git-mergetool config variables when the difftool equivalents have not been defined". Until now, this was not the case for "difftool.prompt". Signed-off-by: Sebastian Schuberth --- git-difftool--helper.sh | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/git-difftool--helper.sh b/git-difftool--helper.sh index 57e8e32..3438aba 100755 --- a/git-difftool--helper.sh +++ b/git-difftool--helper.sh @@ -12,7 +12,8 @@ TOOL_MODE=diff # difftool.prompt controls the default prompt/no-prompt behavior # and is overridden with $GIT_DIFFTOOL*_PROMPT. should_prompt () { - prompt=$(git config --bool difftool.prompt || echo true) + prompt_merge=$(git config --bool mergetool.prompt || echo true) + prompt=$(git config --bool difftool.prompt || echo $prompt_merge) if test "$prompt" = true; then test -z "$GIT_DIFFTOOL_NO_PROMPT" else -- 1.6.6.265.ga0f40 ``` ## David Aguilar, 2010-01-22 21:24 Subject: Re: [PATCHv2] Make difftool.prompt fall back to mergetool.prompt Message-ID: <7B0478B5-CEE2-4207-8268-B0B338F88449@gmail.com> URL: https://gitlist.dev/e/7B0478B5-CEE2-4207-8268-B0B338F88449%40gmail.com In-Reply-To: <7vr5pivx70.fsf@alter.siamese.dyndns.org> ``` On Jan 22, 2010, at 9:58 AM, Junio C Hamano wrote: > Sebastian Schuberth writes: > >> The documentation states that "git-difftool falls back to git- >> mergetool >> config variables when the difftool equivalents have not been >> defined". >> Until now, this was not the case for "difftool.prompt". >> >> Signed-off-by: Sebastian Schuberth > > David, does this look ok? It seems to match the discussion between > you I > saw yesterday, but just checking to make sure (if "yes" please Ack). Acked-by: David Aguilar I would like to see a testcase to ensure we don't break it in the future, though. Thanks Sebastian, Junio. Sebastian, I you don't send a test I'll try to send one tonight. > > Is it Ok that today I see only this patch not two-patch series on the > topic (iow, has the other patch been withdrawn)? I didn't folow the > discussion very closely. > > Thanks. > >> --- >> git-difftool--helper.sh | 3 ++- >> 1 files changed, 2 insertions(+), 1 deletions(-) >> >> diff --git a/git-difftool--helper.sh b/git-difftool--helper.sh >> index 57e8e32..3438aba 100755 >> --- a/git-difftool--helper.sh >> +++ b/git-difftool--helper.sh >> @@ -12,7 +12,8 @@ TOOL_MODE=diff >> # difftool.prompt controls the default prompt/no-prompt behavior >> # and is overridden with $GIT_DIFFTOOL*_PROMPT. >> should_prompt () { >> - prompt=$(git config --bool difftool.prompt || echo true) >> + prompt_merge=$(git config --bool mergetool.prompt || echo true) >> + prompt=$(git config --bool difftool.prompt || echo >> $prompt_merge) >> if test "$prompt" = true; then >> test -z "$GIT_DIFFTOOL_NO_PROMPT" >> else >> -- >> 1.6.6.265.ga0f40 ``` ## Sebastian Schuberth, 2010-01-22 23:31 Subject: Re: [PATCHv2] Make difftool.prompt fall back to mergetool.prompt Message-ID: URL: https://gitlist.dev/e/bdca99241001221531t24afc87cydf080f432cfbc51a%40mail.gmail.com In-Reply-To: <7B0478B5-CEE2-4207-8268-B0B338F88449@gmail.com> ``` On Fri, Jan 22, 2010 at 22:24, David Aguilar wrote: > Sebastian, I you don't send a test I'll try to send one tonight. I'd be glad if you could propose a testcase. Thanks. -- Sebastian Schuberth ```