Jeff King schrieb: > @@ -231,6 +232,8 @@ static int run_command(struct cmd_struct *p, int argc, const char **argv) > use_pager = check_pager_config(p->cmd); > if (use_pager == -1 && p->option & USE_PAGER) > use_pager = 1; > + if (use_pager == 1 && p->option & FORBID_PAGER) > + use_pager = 0; > commit_pager_choice(); > > if (p->option & NEED_WORK_TREE) > @@ -286,7 +289,7 @@ static void handle_internal_command(int argc, const char **argv) > { "count-objects", cmd_count_objects, RUN_SETUP }, > { "describe", cmd_describe, RUN_SETUP }, > { "diff", cmd_diff }, > - { "diff-files", cmd_diff_files, RUN_SETUP }, > + { "diff-files", cmd_diff_files, RUN_SETUP | FORBID_PAGER }, Every now and then I want to use 'git -p diff-files', and I think that is a valid use-case. But your suggested patch seems to forbid the pager even in this case. :-( -- Hannes -- 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