Skip to content
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.

Sign up for GitHub

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

Jump to bottom

time zone data: 2015b #201

Merged
merged 1 commit into from
Apr 10, 2015
Merged

time zone data: 2015b #201

merged 1 commit into from
Apr 10, 2015

Conversation

vmeurisse
Copy link
Contributor

No description provided.

mattjohnsonpint
mattjohnsonpint reviewed Apr 7, 2015
View reviewed changes
data/unpacked/latest.json
@@ -10537,15 +10534,15 @@
{
"name": "America/Cayman",
"abbrs": [
"KMT",
"CMT",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't in the 2015b sources. Where did it come from?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This changed is caused by America/Cayman now being a link to America/Panama
eggert/tz@72a3ac9#diff-9c77a7aa95323ac0135649098471287eL2600

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vmeurisse - yep, looks like that was the reason. Thanks.

@mattjohnsonpint
Copy link
Contributor

I see a few other discrepancies that aren't in 2015b. I'm comparing against:
http://mm.icann.org/pipermail/tz-announce/2015-March/000029.html
https://github.com/eggert/tz/tree/2015b
http://www.iana.org/time-zones/repository/releases/tzdata2015b.tar.gz

@vmeurisse Can you please start clean and rebase your branch? Thanks.

@vmeurisse
Copy link
Contributor Author

@mj1856 I retried with a clean branch and both zdumb and zic compiled from https://github.com/eggert/tz/tree/2015b but I still get the same results

@mattjohnsonpint
Copy link
Contributor

Looks good. Thanks for the explaination.

mattjohnsonpint added a commit that referenced this pull request Apr 10, 2015
@mattjohnsonpint
Merge pull request #201 from vmeurisse/2015b
532cf3e 
time zone data: 2015b
@mattjohnsonpint mattjohnsonpint merged commit 532cf3e into moment:develop Apr 10, 2015
@timrwood timrwood mentioned this pull request May 30, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vmeurisse @mattjohnsonpint