Skip to content
New issue

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

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

Already on GitHub? # to your account

Support relative path for ICU_DAT_FILE_PATH on iOS/tvOS/Catalyst #87813

Merged
merged 1 commit into from
Jun 20, 2023

Conversation

akoeplinger
Copy link
Member

We consider it relative to the app bundle root.

Required for dotnet/macios#17877 and NativeAOT scenarios.

We consider it relative to the app bundle root.

Required for dotnet/macios#17877 and NativeAOT scenarios.
@ghost
Copy link

ghost commented Jun 20, 2023

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

Issue Details

We consider it relative to the app bundle root.

Required for dotnet/macios#17877 and NativeAOT scenarios.

Author: akoeplinger
Assignees: -
Labels:

area-System.Globalization

Milestone: -

@ghost
Copy link

ghost commented Jun 20, 2023

Tagging subscribers to 'os-ios': @steveisok, @akoeplinger, @kotlarmilos
See info in area-owners.md if you want to be subscribed.

Issue Details

We consider it relative to the app bundle root.

Required for dotnet/macios#17877 and NativeAOT scenarios.

Author: akoeplinger
Assignees: akoeplinger
Labels:

area-System.Globalization, os-ios, os-tvos, os-maccatalyst

Milestone: -

@akoeplinger akoeplinger merged commit f450f49 into dotnet:main Jun 20, 2023
@akoeplinger akoeplinger deleted the icu-data-relative branch June 20, 2023 15:48
rolfbjarne added a commit to rolfbjarne/xamarin-macios that referenced this pull request Jun 20, 2023
We can fix this better once this fix reaches us:

    dotnet/runtime#87813

because then we can set the ICU data file at build time (to a relative path).
rolfbjarne added a commit to rolfbjarne/xamarin-macios that referenced this pull request Jun 22, 2023
We can fix this better once this fix reaches us:

    dotnet/runtime#87813

because then we can set the ICU data file at build time (to a relative path).
@ghost ghost locked as resolved and limited conversation to collaborators Jul 20, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants