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

Disable objc_msgSend stubs in clang #89932

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented Aug 3, 2023

Applies the same fix as xamarin/xamarin-macios#16231.

We recently started seeing the same issue in our runtime builds, see #89925. This is because the AzDO build machines use Xcode 14 now but the Helix machines are still using Xcode 13 so linking an Xcode-14-built static library fails.

This happens in e.g. libSystem.Native.a or libSystem.Globalization.Native.a (the two libraries where we're using Objective-C).

Fixes #89925

@ghost
Copy link

ghost commented Aug 3, 2023

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

Applies the same fix as xamarin/xamarin-macios#16231.

We recently started seeing the same issue in our runtime builds, see #89925. This is because the AzDO build machines use Xcode 14 but the Helix machines are still using Xcode 13 so linking an Xcode-14-built static library fails.

Fixes #89925

Author: akoeplinger
Assignees: akoeplinger
Labels:

area-Infrastructure-coreclr

Milestone: -

@akoeplinger
Copy link
Member Author

/azp run runtime-ioslikesimulator

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ghost
Copy link

ghost commented Aug 3, 2023

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

Issue Details

Applies the same fix as xamarin/xamarin-macios#16231.

We recently started seeing the same issue in our runtime builds, see #89925. This is because the AzDO build machines use Xcode 14 but the Helix machines are still using Xcode 13 so linking an Xcode-14-built static library fails.

Fixes #89925

Author: akoeplinger
Assignees: akoeplinger
Labels:

area-Infrastructure-libraries

Milestone: -

@akoeplinger
Copy link
Member Author

/azp run runtime-ioslikesimulator

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger
Copy link
Member Author

/azp run runtime-ioslikesimulator

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Applies the same fix as xamarin/xamarin-macios#16231.

We recently started seeing the same issue in our runtime builds, see dotnet#89925.
This is because the AzDO build machines use Xcode 14 but the Helix machines are still using Xcode 13 so linking an Xcode-14-built static library fails.

Fixes dotnet#89925
@akoeplinger
Copy link
Member Author

/azp run runtime-ioslikesimulator

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jakobbotsch
Copy link
Member

jakobbotsch commented Aug 8, 2023

@akoeplinger I am getting the following build error on my Mac with this PR applied:

  CMake Error at /Users/jakobbotsch/dev/dotnet/runtime/src/native/libs/System.Native/CMakeLists.txt:99 (set_source_files_properties):
    set_source_files_properties called with incorrect number of arguments.
  
  
  CMake Error at /Users/jakobbotsch/dev/dotnet/runtime/src/native/libs/System.Globalization.Native/CMakeLists.txt:97 (set_source_files_properties):
    set_source_files_properties called with incorrect number of arguments.

Any idea what could be the problem? The build works if I revert this (actually no, I get to almost the end of the build but then see a linker error).

Edit: After updating my Mac from Monterey to Ventura and then xcode from 14.2 to 14.3.1 things seem to build fine again, so feel free to disregard this.

akoeplinger added a commit to akoeplinger/runtime that referenced this pull request Aug 9, 2023
We were missing the objc_msgsend fix from dotnet#89932 in the mono build.
Also fix an issue where CMake would complain about wrong number of arguments when CLR_CMAKE_COMMON_OBJC_FLAGS is empty, we need put the variable in quotes.
@akoeplinger
Copy link
Member Author

@jakobbotsch thanks for reporting, that is actually a real issue when the variable is empty. I fixed it in #90217

akoeplinger added a commit that referenced this pull request Aug 9, 2023
…#90217)

We were missing the objc_msgsend fix from #89932 in the mono build.

Also fix an issue where CMake would complain about wrong number of arguments when CLR_CMAKE_COMMON_OBJC_FLAGS is empty, we need put the variable in quotes.
akoeplinger added a commit to akoeplinger/runtime that referenced this pull request Aug 25, 2023
akoeplinger added a commit to akoeplinger/runtime that referenced this pull request Aug 25, 2023
akoeplinger added a commit that referenced this pull request Aug 29, 2023
akoeplinger added a commit that referenced this pull request Aug 29, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[mono][iossimulator] Undefined globalization symbols for iossimulator-arm64
4 participants