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

Linux Arm64 TimeZoneInfo Duplicate DisplayNames #64111

Open
mdh1418 opened this issue Jan 21, 2022 · 5 comments
Open

Linux Arm64 TimeZoneInfo Duplicate DisplayNames #64111

mdh1418 opened this issue Jan 21, 2022 · 5 comments
Labels
arch-arm64 area-System.DateTime enhancement Product code improvement that does NOT require public API changes/additions
Milestone
Future

Comments

@mdh1418
Copy link
Member

mdh1418 commented Jan 21, 2022

Through work to ensure no duplicate display names for timezones on Android, it was discovered that there are duplicates on Linux Arm64.

tzCollection: id America/Punta_Arenas has DisplayName (UTC-03:00) GMT
tzCollection: id America/Nuuk has DisplayName (UTC-03:00) GMT

tzCollection: id Europe/Astrakhan has DisplayName (UTC+04:00) GMT
tzCollection: id Europe/Ulyanovsk has DisplayName (UTC+04:00) GMT
tzCollection: id Europe/Saratov has DisplayName (UTC+04:00) GMT

tzCollection: id Asia/Tomsk has DisplayName (UTC+07:00) GMT
tzCollection: id Asia/Barnaul has DisplayName (UTC+07:00) GMT

On Linux musl x64 and Android x64, for example, the IDs have different display names for

tzCollection: id America/Punta_Arenas has DisplayName (UTC-03:00) Punta Arenas Time

tzCollection: id Europe/Astrakhan has DisplayName (UTC+04:00) Astrakhan Time
tzCollection: id Europe/Ulyanovsk has DisplayName (UTC+04:00) Ulyanovsk Time
tzCollection: id Europe/Saratov has DisplayName (UTC+04:00) Saratov Time

tzCollection: id Asia/Tomsk has DisplayName (UTC+07:00) Tomsk Time
tzCollection: id Asia/Barnaul has DisplayName (UTC+07:00) Barnaul Time

Full list of GetSystemTimeZone IDs and DIsplayNames https://gist.github.com/mdh1418/a2dca0098fea5af098fcee92639b6515
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Jan 21, 2022
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Jan 23, 2022

Tagging subscribers to this area: @dotnet/area-system-globalization
See info in area-owners.md if you want to be subscribed.

Issue Details

Through work to ensure no duplicate display names for timezones on Android, it was discovered that there are duplicates on Linux Arm64.

tzCollection: id America/Punta_Arenas has DisplayName (UTC-03:00) GMT
tzCollection: id America/Nuuk has DisplayName (UTC-03:00) GMT

tzCollection: id Europe/Astrakhan has DisplayName (UTC+04:00) GMT
tzCollection: id Europe/Ulyanovsk has DisplayName (UTC+04:00) GMT
tzCollection: id Europe/Saratov has DisplayName (UTC+04:00) GMT

tzCollection: id Asia/Tomsk has DisplayName (UTC+07:00) GMT
tzCollection: id Asia/Barnaul has DisplayName (UTC+07:00) GMT

On Linux musl x64 and Android x64, for example, the IDs have different display names for

tzCollection: id America/Punta_Arenas has DisplayName (UTC-03:00) Punta Arenas Time

tzCollection: id Europe/Astrakhan has DisplayName (UTC+04:00) Astrakhan Time
tzCollection: id Europe/Ulyanovsk has DisplayName (UTC+04:00) Ulyanovsk Time
tzCollection: id Europe/Saratov has DisplayName (UTC+04:00) Saratov Time

tzCollection: id Asia/Tomsk has DisplayName (UTC+07:00) Tomsk Time
tzCollection: id Asia/Barnaul has DisplayName (UTC+07:00) Barnaul Time

Full list of GetSystemTimeZone IDs and DIsplayNames https://gist.github.com/mdh1418/a2dca0098fea5af098fcee92639b6515

Author: mdh1418
Assignees: -
Labels:

area-System.Globalization, untriaged
Milestone: -

@ghost ghost added this to Untriaged in ML, Extensions, Globalization, etc, POD. Jan 23, 2022
@tarekgh tarekgh added area-System.Runtime and removed area-System.Globalization untriaged New issue has not been triaged by the area owner labels Jan 23, 2022
@ghost ghost removed this from Untriaged in ML, Extensions, Globalization, etc, POD. Jan 23, 2022
@ghost
Copy link

ghost commented Jan 23, 2022

Tagging subscribers to this area: @dotnet/area-system-runtime
See info in area-owners.md if you want to be subscribed.

Issue Details

Through work to ensure no duplicate display names for timezones on Android, it was discovered that there are duplicates on Linux Arm64.

tzCollection: id America/Punta_Arenas has DisplayName (UTC-03:00) GMT
tzCollection: id America/Nuuk has DisplayName (UTC-03:00) GMT

tzCollection: id Europe/Astrakhan has DisplayName (UTC+04:00) GMT
tzCollection: id Europe/Ulyanovsk has DisplayName (UTC+04:00) GMT
tzCollection: id Europe/Saratov has DisplayName (UTC+04:00) GMT

tzCollection: id Asia/Tomsk has DisplayName (UTC+07:00) GMT
tzCollection: id Asia/Barnaul has DisplayName (UTC+07:00) GMT

On Linux musl x64 and Android x64, for example, the IDs have different display names for

tzCollection: id America/Punta_Arenas has DisplayName (UTC-03:00) Punta Arenas Time

tzCollection: id Europe/Astrakhan has DisplayName (UTC+04:00) Astrakhan Time
tzCollection: id Europe/Ulyanovsk has DisplayName (UTC+04:00) Ulyanovsk Time
tzCollection: id Europe/Saratov has DisplayName (UTC+04:00) Saratov Time

tzCollection: id Asia/Tomsk has DisplayName (UTC+07:00) Tomsk Time
tzCollection: id Asia/Barnaul has DisplayName (UTC+07:00) Barnaul Time

Full list of GetSystemTimeZone IDs and DIsplayNames https://gist.github.com/mdh1418/a2dca0098fea5af098fcee92639b6515

Author: mdh1418
Assignees: -
Labels:

area-System.Runtime
Milestone: -

@tarekgh tarekgh added this to the Future milestone Jan 23, 2022
@tarekgh tarekgh added the enhancement Product code improvement that does NOT require public API changes/additions label Jan 23, 2022
@tarekgh
Copy link
Member

tarekgh commented Jan 23, 2022

@mattjohnsonpint do you think there is a way we can distinguish the display names for the listed zones?

@mattjohnsonpint
Copy link
Contributor

I don't quite understand why it would matter that it's Arm64. Could it just be that the machine had an outdated version of ICU or its data?

That said, it's really bad that the display name is "GMT". That shouldn't ever be there. If for some reason we can't get a true display name, we should fall back to using the IANA ID, for example "(UTC+04:00) Europe/Astrakhan"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
arch-arm64 area-System.DateTime enhancement Product code improvement that does NOT require public API changes/additions
Projects
None yet
Milestone
Future
Development

No branches or pull requests
5 participants
@jeffhandley @mattjohnsonpint @jeffschwMSFT @tarekgh @mdh1418