From: Stephen & Linda Smith <ischis2@cox.net>
Date: Thu, 10 Jan 2019 01:50:26 GMT
Subject: Re: [PATCH 3/3] t0006-date.sh: add `human` date format tests.
Message-ID: <2249216.RGRfnxe1O0@thunderbird>
In-Reply-To: <42e521e3-8a12-17c9-cb7f-bfba226ca126@kdbg.org>

On Tuesday, January 8, 2019 11:58:29 PM MST Johannes Sixt wrote:
> But notice that the value of $TODAY_REGEX contains blanks.
> 
> In this line
> 
> check_human_date "$(($(date +%s)-18000)) +0200" $TODAY_REGEX
> 
> the value of $TODAY_REGEX is substituted and then the value is split
> into fields at the blanks because the expansion is not quoted.
> 
> As a consequence, function check_human_date considers only the first
> part of $TODAY_REGEX, i.e. 'A-Z][a-z][a-z]' (which is parameter $2), but
> ignores everything else (because it does not use $3 or $4).
> 
> -- Hannes

I hadn't understood your original comment, but now i understand.   Will fix.