Re: [PATCH] t/helper: 'test-chmtime (--get|-g)' to print only the mtime

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 9, 2018, 02:35 UTC
Message-ID: <xmqq1sfp6r9n.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20180406221947.28402-1-ungureanupaulsebastian@gmail.com>

Paul-Sebastian Ungureanu <ungureanupaulsebastian@gmail.com> writes:

Show 24 quoted lines

> Compared to 'test-chmtime -v +0 file' which prints the mtime and
> and the file name, 'test-chmtime --get file' displays only the mtime.
> If it is used in combination with (+|=|=+|=-|-)seconds, it changes
> and prints the new value.
>
> 	test-chmtime -v +0 file | sed 's/[^0-9].*$//'
>
> is now equivalent to:
>
> 	test-chmtime --get file
>
> Signed-off-by: Paul-Sebastian Ungureanu <ungureanupaulsebastian@gmail.com>
> ---
>  t/helper/test-chmtime.c              | 44 +++++++++++++++++++++++-----
>  t/t2022-checkout-paths.sh            |  4 +--
>  t/t3404-rebase-interactive.sh        |  2 +-
>  t/t3510-cherry-pick-sequence.sh      |  4 +--
>  t/t4200-rerere.sh                    |  8 ++---
>  t/t5000-tar-tree.sh                  |  2 +-
>  t/t6022-merge-rename.sh              | 25 +++++++---------
>  t/t6501-freshen-objects.sh           |  6 ++--
>  t/t7508-status.sh                    |  4 +--
>  t/t7701-repack-unpack-unreachable.sh |  6 ++--
>  10 files changed, 63 insertions(+), 42 deletions(-)

Thanks both for suggesting and implementing an obvious improvement and updating many places that are helped by it.

Will queue. .