Am 12/13/2011 23:35, schrieb Joe Ratterman: > +test_expect_success TTY 'no pager with -P' ' > + rm -f paginated.out || > + cleanup_fail && > + > + test_terminal git -P log && > + ! test -e paginated.out > +' > + > test_expect_success TTY 'no pager with --no-pager' ' > rm -f paginated.out || > cleanup_fail && What kind of bogosity do I see in the context of this hunk (and in the new text as well, but it is not entirely your fault, Joe, since you obviously have only copied an existing test snippet)? Wouldn't rm -f always succeed under normal circumstances, and then the rest of the test would be skipped? This use of || was introduced by fdf1bc48 (t7006: guard cleanup with test_expect_success). -- 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