Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Port TimeZoneInfo cleanups from CoreCLR #2388

Closed
jkotas opened this issue Dec 19, 2016 · 9 comments
Closed

Port TimeZoneInfo cleanups from CoreCLR #2388

jkotas opened this issue Dec 19, 2016 · 9 comments

Comments

@jkotas
Copy link
Member

jkotas commented Dec 19, 2016

@jkotas
Copy link
Member Author

jkotas commented Dec 19, 2016

cc @tarekgh @DnlHarvey

@justinvp Would you like to take care of these ports?

@tarekgh
Copy link
Member

tarekgh commented Dec 19, 2016

@DnlHarvey could you please help with the port?
I assigned the issue to you, let me know if you can't do it so I can try to schedule doing it.

@justinvp
Copy link
Contributor

I can take care of porting these. I might not get to it for a few days, though.

@tarekgh
Copy link
Member

tarekgh commented Dec 20, 2016

thanks @justinvp

@jamesqo
Copy link
Contributor

jamesqo commented Dec 23, 2016

Maybe this can be closed? The associated PR was merged.

@tarekgh tarekgh closed this as completed Dec 23, 2016
@justinvp
Copy link
Contributor

dotnet/coreclr#8512 still needs to be ported in a separate PR

@tarekgh tarekgh reopened this Dec 23, 2016
@tarekgh
Copy link
Member

tarekgh commented Dec 23, 2016

I re-opened it but it would be better to track the porting on corert branch.

@justinvp
Copy link
Contributor

it would be better to track the porting on corert branch.

Not sure what you mean. This issue is in the corert repo for work that needs to happen in corert.

dotnet/coreclr#8512 is the only remaining item to port. It's not a straightforward port like the others as it requires some minor refactoring due to differences in the implementation. I started working on it.

@tarekgh
Copy link
Member

tarekgh commented Dec 23, 2016

@justinvp never mind.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants