Skip to content

[dotnet] Use correct relative path for install_name_tool on macOS - #9804

Merged
rolfbjarne merged 2 commits into
dotnet:mainfrom
filipnavara:fix-lib-path
Oct 8, 2020
Merged

[dotnet] Use correct relative path for install_name_tool on macOS#9804
rolfbjarne merged 2 commits into
dotnet:mainfrom
filipnavara:fix-lib-path

Conversation

@filipnavara

Copy link
Copy Markdown
Member

No description provided.

@rolfbjarne rolfbjarne added macOS Issues affecting macOS community Community contribution ❤ dotnet An issue or pull request related to .NET (6) run-dotnet-tests Run all the .NET tests labels Oct 7, 2020
Comment thread dotnet/targets/Xamarin.Shared.Sdk.targets Outdated
@spouliot spouliot added this to the .NET 6 milestone Oct 7, 2020

@dalexsoto dalexsoto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After Rolf's 👍

@mandel-macaque mandel-macaque left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 after rolf commit is added.

@rolfbjarne

Copy link
Copy Markdown
Member

build

@monojenkins

Copy link
Copy Markdown
Contributor

Build success
Provisioning succeeded
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
ℹ️ Generator Diff (please review changes)
Test run succeeded

@filipnavara

Copy link
Copy Markdown
Member Author

I'm working on a better fix for this problem. Looks like the .NET Core system libraries now have proper identity with @rpath prefix so the whole _UpdateDynamicLibraryId step could be removed. The LinkNativeCode however specifies incorrect -rpath for macOS builds so that still needs to be fixed.

This PR is still probably worth it as a stopgap solution and because of the removal of hard-coded MonoBundle from one place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Community contribution ❤ dotnet An issue or pull request related to .NET (6) macOS Issues affecting macOS run-dotnet-tests Run all the .NET tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants