format-patch: ignore diff.submodule setting

diff.submodule when set to log produces output which git-am cannot
handle. Ignore this setting when generating patch output.

Signed-off-by: Doug Kelly <dougk.ff7@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/builtin/log.c b/builtin/log.c
index 734aab3a73185964d18c82fe5eb31008bbc409b7..cb14db48987247aae85f4891bd8fec324f709c56 100644 (file)
--- a/builtin/log.c
+++ b/builtin/log.c
@@ -705,7 +705,7 @@ static int git_format_config(const char *var, const char *value, void *cb)
                return 0;
        }
        if (!strcmp(var, "diff.color") || !strcmp(var, "color.diff") ||
                return 0;
        }
        if (!strcmp(var, "diff.color") || !strcmp(var, "color.diff") ||

-           !strcmp(var, "color.ui")) {
+           !strcmp(var, "color.ui") || !strcmp(var, "diff.submodule")) {

                return 0;
        }
        if (!strcmp(var, "format.numbered")) {
                return 0;
        }
        if (!strcmp(var, "format.numbered")) {

diff --git a/t/t4255-am-submodule.sh b/t/t4255-am-submodule.sh
index 450d26136a0555822703ff4deb9ff3bb1326936c..0ba8194403f6740807cbd63d1bda8a69091e3d6b 100755 (executable)
--- a/t/t4255-am-submodule.sh
+++ b/t/t4255-am-submodule.sh
@@ -80,12 +80,12 @@ test_expect_success 'diff.submodule unset with extra file' '
        run_test $THIRD second-submodule
 '
 
        run_test $THIRD second-submodule
 '
 

-test_expect_failure 'diff.submodule=log' '
+test_expect_success 'diff.submodule=log' '

        test_config diff.submodule log &&
        run_test $SECOND first-submodule
 '
 
        test_config diff.submodule log &&
        run_test $SECOND first-submodule
 '
 

-test_expect_failure 'diff.submodule=log with extra file' '
+test_expect_success 'diff.submodule=log with extra file' '

        test_config diff.submodule log &&
        run_test $THIRD second-submodule
 '
        test_config diff.submodule log &&
        run_test $THIRD second-submodule
 '