Skip to content

hotfix (tester) - Easter Island & Haiti Windows time zone#4317

Merged
Hackerpilot merged 1 commit intomasterfrom
unknown repository
May 13, 2016
Merged

hotfix (tester) - Easter Island & Haiti Windows time zone#4317
Hackerpilot merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented May 13, 2016

The windows tester systematically fails because a new windows time zone entry was added for the Easter Island.

https://blogs.technet.microsoft.com/dst2007/2016/03/23/time-zone-update-for-chile-and-haiti/

This PR fixes the two functions involved. (tzDatabaseNameToWindowsTZName and windowsTZNameToTZDatabaseName). They're gonna be deprecated in next rlz but they still need to be maintained.

reference for the names:

@ghost ghost changed the title hotfix (tester) - Easter Island Windows time zone hotfix (tester) - Easter Island & Haiti Windows time zone May 13, 2016
@ghost
Copy link
Author

ghost commented May 13, 2016

win_32_64 passed, we need auto-merge toggled on ASAP thx.

@ghost
Copy link
Author

ghost commented May 13, 2016

linux_64_64 failure is unrelated, the demand is still valid.

@Hackerpilot
Copy link
Contributor

Auto-merge toggled on

@Hackerpilot Hackerpilot merged commit 772f026 into dlang:master May 13, 2016
@ghost ghost deleted the fix_wtz branch May 13, 2016 07:58
@ghost
Copy link
Author

ghost commented May 13, 2016

@MartinNowak don't forget to cherry pick in stable ;)

9rnsr pushed a commit to 9rnsr/phobos that referenced this pull request May 13, 2016
hotfix (tester) - Easter Island & Haiti Windows time zone
@9rnsr
Copy link
Contributor

9rnsr commented May 13, 2016

I opened #4321 to cherry-pick the change into stable branch.

9rnsr added a commit that referenced this pull request May 13, 2016
Cherry-pick commtis in #4317 from master to stable branch
@ghost
Copy link
Author

ghost commented May 13, 2016

Nice. It' important for me to fix the hot stuffs before I leave. My daughter gets a bit angry otherwise...

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants