Tay Ray Chuan wrote: > --- a/builtin/log.c > +++ b/builtin/log.c > @@ -1159,6 +1159,8 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix) > > if (!use_stdout) > output_directory = set_outdir(prefix, output_directory); > + else > + setup_pager(); Since it can be disabled with [pager] format-patch = false this looks reasonable to me. Maybe a test to that effect would help, something like this? test_expect_success TTY 'format-patch --stdout paginates' ' rm -f pager_used && ( GIT_PAGER="wc >pager_used" && export GIT_PAGER && test_terminal git format-patch --stdout --all ) && test_path_is_file pager_used ' test_expect_success TTY 'format-patch --stdout pagination can be disabled' ' rm -f pager_used && ( GIT_PAGER="wc >pager_used" && export GIT_PAGER && test_terminal git --no-pager format-patch --stdout --all && test_terminal git -c "pager.format-patch=false" format-patch --stdout --all ) && test_path_is_missing pager_used && test_path_is_missing .git/pager_used ' -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html