diff mbox series

[v4,08/10] t/helper: allow chmtime to print verbosely without modifying mtime

Message ID 548ac777a9dd317c1def68a9b5569bbf889b20fe.1681452028.git.ps@pks.im (mailing list archive)
State Accepted
Commit 19a3a7bde9164c86353c944e939a1dcb537993f9
Headers show
Series [v4,01/10] midx: fix segfault with no packs and invalid preferred pack | expand

Commit Message

Patrick Steinhardt April 14, 2023, 6:02 a.m. UTC 
The `test-tool chmtime` helper allows us to both read and modify the
modification time of files. But while it is possible to only read the
mtimes of a file via `--get`, it is not possible to read the mtimes
and report them together with their respective file paths via the
`--verbose` flag without also modifying the mtime at the same time.

Fix this so that it is possible to call `test-tool chmtime --verbose
<files>...` without modifying any mtimes.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 t/helper/test-chmtime.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/t/helper/test-chmtime.c b/t/helper/test-chmtime.c
index dc28890a18..0e5538833a 100644
--- a/t/helper/test-chmtime.c
+++ b/t/helper/test-chmtime.c
@@ -94,7 +94,7 @@  int cmd__chmtime(int argc, const char **argv)
 	if (timespec_arg(argv[i], &set_time, &set_eq)) {
 		++i;
 	} else {
-		if (get == 0) {
+		if (get == 0 && verbose == 0) {
 			fprintf(stderr, "Not a base-10 integer: %s\n", argv[i] + 1);
 			goto usage;
 		}