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.

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

[mono] iOS Simulator builds will pick up a system ICU in some circumstances #50365

Closed
CoffeeFlux opened this issue Mar 29, 2021 · 3 comments · Fixed by #73467
Closed

[mono] iOS Simulator builds will pick up a system ICU in some circumstances #50365

CoffeeFlux opened this issue Mar 29, 2021 · 3 comments · Fixed by #73467
Assignees
Milestone

Comments

@CoffeeFlux
Copy link
Contributor

CoffeeFlux commented Mar 29, 2021

Error:

  ld: building for iOS Simulator, but linking in dylib built for macOS, file '/usr/local/opt/icu4c/lib/libicui18n.dylib' for architecture x86_64
  clang: error: linker command failed with exit code 1 (use -v to see invocation)
  ninja: build stopped: subcommand failed.

Env vars set (homebrew-recommended):

# icu4c is keg-only
export PATH="/usr/local/opt/icu4c/bin:$PATH"
export PATH="/usr/local/opt/icu4c/sbin:$PATH"
export LDFLAGS="-L/usr/local/opt/icu4c/lib $LDFLAGS"
export CPPFLAGS="-I/usr/local/opt/icu4c/include $CPPFLAGS"
export PKG_CONFIG_PATH="/usr/local/opt/icu4c/lib/pkgconfig:$PKG_CONFIG_PATH"

It should probably just never check the system, given I would not expect there to ever be a system install of iOS ICU builds.

@CoffeeFlux CoffeeFlux added this to the 6.0.0 milestone Mar 29, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Mar 29, 2021
@CoffeeFlux CoffeeFlux changed the title [mono] iOS Simulator builds will pick up a system ICU on your PATH [mono] iOS Simulator builds will pick up a system ICU in some circumstances Mar 29, 2021
@CoffeeFlux
Copy link
Contributor Author

cc: @steveisok

@SamMonoRT SamMonoRT removed the untriaged New issue has not been triaged by the area owner label Jun 23, 2021
@steveisok
Copy link
Member

/cc @akoeplinger if it's still relevant.

@steveisok steveisok modified the milestones: 6.0.0, Future Jul 12, 2021
@akoeplinger akoeplinger modified the milestones: Future, 7.0.0 Jul 14, 2021
@steveisok
Copy link
Member

@akoeplinger Please check to see if this still happens. If not, close.

akoeplinger added a commit to akoeplinger/runtime that referenced this issue Aug 5, 2022
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Aug 5, 2022
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Aug 5, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Sep 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants