Signed-off-by: Ramsay Jones <>

Hi Jens,

commit 61b6a633 ("commit -v: strip diffs and submodule shortlogs
from the commit message", 19-11-2013) in 'pu' fails the new test
it added to t7507.

I didn't spend too long looking at the problem, so take this patch
as nothing more than a quick suggestion for a possible solution! :-P
[The err file contained something like: "There was a problem with the
editor '"$FAKE_EDITOR"'"].

Having said that, this fixes it for me ...

Ramsay Jones

 t/ | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/t/ b/t/
index 09c1150..49cfb3c 100755
--- a/t/
+++ b/t/
@@ -79,7 +79,8 @@ test_expect_success 'submodule log is stripped out too with 
-v' '
                echo "more" >>file &&
                git commit -a -m "submodule commit"
        ) &&
-       GIT_EDITOR=cat test_must_fail git commit -a -v 2>err &&
+       test_set_editor cat &&
+       test_must_fail git commit -a -v 2>err &&
        test_i18ngrep "Aborting commit due to empty commit message." err
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to