Fix strflocaltime on daylight saving time values

[wjlroe]

This PR changes the way time values get serialised and deserialised to the internal format when passing them between stages. Previously the internal value didn't retain the dst value so time values would lose that field and when printed using strflocaltime, the fact the value was in DST was lost. This seems to me to be the only way to have strflocaltime printing the correct time zone information on values during a daylight saving time.

The following behaviour demonstrates how the daylight savings value gets
lost when a time value gets turned into an internal array of numbers and
then back into a (struct tm):

    $ TZ=Europe/London ./jq -r -n '1504803635 | strflocaltime("%Y-%m-%d %H:%M:%S (%Z)")'
    2017-09-07 18:00:35 (GMT)

After this change however, this works as expected:

    $ TZ=Europe/London ./jq -r -n '1504803635 | strflocaltime("%Y-%m-%d %H:%M:%S (%Z)")'
    2017-09-07 18:00:35 (BST)

(the only difference is in the %Z value - "BST" instead of the incorrect "GMT").

[coveralls]

Coverage increased (+0.2%) to 84.355% when pulling d25af32 on wjlroe:fix-dst-strflocaltime into a17dd32 on stedolan:master.

[pkoppstein]

@wtlangford - I cannot say whether this PR fixes all TZ-related bugs in jq, but I hope that you will keep in mind that the serious TZ-related bugs in jq are "silent" and as such really deserve to be prioritized, even over a release of jq 1.7.

If you're too busy to attend to these "silent" TZ-related bugs yourself, perhaps we could recruit some volunteers to review this particular PR, but if we do so, would you be able at least to try to "pull" the PR if the reviewers attest to its being an improvement? Thanks.

[itchyny]

Issue: #1912.

[DehanLUO]

Branch 439cf72 works for me, thx.

[trantor]

@nicowilliams was this fixed by the recent locale-related patches?

[nicowilliams]

@nicowilliams was this fixed by the recent locale-related patches?

No, sadly.

[chrismo]

@nicowilliams was this fixed by the recent locale-related patches?

No, sadly.

I installed 1.7 yesterday and it seemed to have fixed one of my scripts that uses it? /shrug