Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 07/16] t5304: use test_path_is_* instead of "test -f"

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:02:39

Jeff King [off-list ref] writes:
This is slightly more robust (checking "! test -f" would not
notice a directory of the same name, though that is not
likely to happen here). It also makes debugging easier, as
the test script will output a message on failure.

Signed-off-by: Jeff King <redacted>
---
This patch is totally optional. I did it while debugging t5304 (strange
how badly prune works when you accidentally invert the mtime check!)
and figured it might be worth keeping as a cleanup.
Looks sensible; thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help