Re: [PATCH] Fix '--relative-date'

From: Jeff King <peff@peff.net>
Date: Oct 3, 2009, 10:06 UTC
Message-ID: <20091003100615.GC17873@coredump.intra.peff.net>
In-Reply-To: <1254543618-3772-1-git-send-email-j416@1616.se>

On Sat, Oct 03, 2009 at 01:20:18PM +0900, Johan Sageryd wrote:

> This fixes '--relative-date' so that it does not give '0 year, 12
> months', for the interval 360 <= diff < 365.

Thanks. I think this was a regression introduced recently when we fixed the rounding on how years are printed (it used to just say "1 year" because we were close, but now we always round down, so our boundary for "1 year" must match exactly).

I've picked up your patch in my tree with the following test squashed in:

diff --git a/t/t0006-date.sh b/t/t0006-date.sh
index a4d8fa8..75b02af 100755
--- a/t/t0006-date.sh
+++ b/t/t0006-date.sh
@@ -24,6 +24,7 @@ check_show 13000000 '5 months ago'
 check_show 37500000 '1 year, 2 months ago'
 check_show 55188000 '1 year, 9 months ago'
 check_show 630000000 '20 years ago'
+check_show 31449600 '12 months ago'
 
 check_parse() {
 	echo "$1 -> $2" >expect

-Peff