New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with 'vanguard' data format including negative DST offsets. #88

Closed
voxik opened this Issue May 25, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@voxik

voxik commented May 25, 2018

In Fedora, there was recently announced this change proposal [1]:

As of tzdata-2018e, the upstream will now default to using the 'vanguard' data format including negative DST offsets.

Because we are using the system tzdata to provide the time zone information, I wonder if the tzinfo is compatible with this updated format. Since the tzinfo-data was already updated to tzdata-2018e, I assume that the answer is "yes, tzinfo supports the new format just fine", but I'd like to double check.

Thx

@voxik voxik changed the title from Compatibility with "'vanguard' data format including negative DST offsets."? to Compatibility with "'vanguard' data format including negative DST offsets." May 25, 2018

@voxik voxik changed the title from Compatibility with "'vanguard' data format including negative DST offsets." to Compatibility with 'vanguard' data format including negative DST offsets. May 25, 2018

@philr

This comment has been minimized.

Member

philr commented May 27, 2018

Yes, tzinfo supports the vanguard format introduced in tzdata 2018d that became the main format in 2018e.

Conversions to and from local time will be accurate with any release supporting system zoneinfo files.

I would recommend using the latest v1.2.5 release though. This includes a fix to allow negative daylight savings time offsets to be derived from zoneinfo files. This sets the TimezonePeriod#std_offset and TimezonePeriod#utc_offset attributes correctly when loading zoneinfo time zones that contain negative DST offsets.

@philr philr added the question label May 27, 2018

@philr philr closed this May 27, 2018

@voxik

This comment has been minimized.

voxik commented May 27, 2018

@philr Thanks a lot for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment