Re: [V3 PULL] hwclock: remove date(1)

["Pádraig Brady" <P@draigBrady.com>]

On 04/03/17 13:56, J William Piggott wrote:
> 
> Everyone,
>  Here's the latest branch for adding parse-date.
> 
> Karel,
> I have changed a few things since the last working branch I sent you.  I
> moved the API from parse-date.h to timeutils.h. Cleaned up the code some
> more and made more style conversion. Added documentation and fixed
> hwclock.8.in.
> 
> I've tested the parser pretty thoroughly now, including using odd input
> listed in bug fixes. It passed for all of them.
> 
> Here are examples of the problem related to the latest gnulib
> parse_datetime() commit; which was actually caused by changes in
> coreutils not gnulib.
> 
> BROKEN
> TZ="Europe/London" date -d 'TZ="Australia/Perth" 2016-08-15 07:00'
> Mon Aug 15 07:00:00 AWST 2016
> 
> CORRECT
> TZ="Europe/London" date -d 'TZ="Australia/Perth" 2016-08-15 07:00'
> Mon Aug 15 00:00:00 BST 2016

Sorry for jumping in here. Are you saying there are issues with
the current code in gnulib?

Note the TZ issue in date(1) was added in coreutils-8.26 with:
http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=v8.25-15-gdf88fce
That was fixed with a minimal fix in coreutils-8.26 in Fedora.
coreutils-8.27 will be released in a few days with a different fix
that incorporated changes in parse_datetime in gnulib.

thanks,
Pádraig