From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: elseifthen@gmx.com
Subject: Re: [V3 PULL] hwclock: remove date(1)
To: =?UTF-8?Q?P=c3=a1draig_Brady?=
,
Karel Zak
References:
Cc: util-linux@vger.kernel.org
From: J William Piggott
Message-ID:
Date: Sun, 5 Mar 2017 09:09:28 -0500
MIME-Version: 1.0
In-Reply-To:
Content-Type: text/plain; charset=windows-1252
List-ID:
On 03/04/2017 08:12 PM, Pádraig Brady wrote:
> 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.
Hello Pádraig, no worries.
> Are you saying there are issues with the current code in gnulib?
Nope, I'm saying that the latest gnulib commit regarding
parse_datetime() (gnulib-4e6e16b), which I intentionally omitted from
the version I based this parse_date() submission upon (gnulib-dd7a871),
does not cause this version of parse_date() to be broken. I'm not making a
comment about the state of coreutils nor gnulib, sorry if I implied that
somehow.
>
> 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
>