Skip to content

Don't force using the system libunwind for source-build - #58641

Merged
eerhardt merged 1 commit into
dotnet:mainfrom
omajid:arpow-dont-hardcode-system-libunwind
Sep 9, 2021
Merged

Don't force using the system libunwind for source-build#58641
eerhardt merged 1 commit into
dotnet:mainfrom
omajid:arpow-dont-hardcode-system-libunwind

Conversation

@omajid

Copy link
Copy Markdown
Member

Instead, use the runtime's normal detection logic to decide what to do.

Fixes: dotnet/source-build#2408

Instead, use the runtime's normal detection logic to decide what to do.
Fixes: dotnet/source-build#2408
@ghost

ghost commented Sep 3, 2021

Copy link
Copy Markdown

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

@ghostghost added the community-contribution Indicates that the PR has been added by a community member label Sep 3, 2021
@omajid

Copy link
Copy Markdown
MemberAuthor

cc @dotnet/source-build-contrib

@MichaelSimonsMichaelSimons 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.

@omajid - this will have to be manually ported to the release/6.0 branch as well.

@janvorlijanvorli 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.

LGTM, thank you!

omajid added a commit to omajid/dotnet-runtime that referenced this pull request Sep 9, 2021
Instead, use the runtime's normal detection logic to decide what to do.
Backport of dotnet#58641 to release/6.0.
See: dotnet/source-build#2408
@omajid

Copy link
Copy Markdown
MemberAuthor

Thanks for the reminder, @MichaelSimons! I opened #58911 for that (simple cherry-pick from main).

@eerhardt
eerhardt merged commit 1dc8dfa into dotnet:mainSep 9, 2021
jeffschwMSFT pushed a commit that referenced this pull request Sep 15, 2021
Instead, use the runtime's normal detection logic to decide what to do.
Backport of #58641 to release/6.0.
See: dotnet/source-build#2408
@ghostghost locked as resolved and limited conversation to collaborators Oct 9, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructurecommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should we stop hardcoding -DCLR_CMAKE_USE_SYSTEM_LIBUNWIND=TRUE in runtime?

5 participants

@omajid@MichaelSimons@janvorli@marek-safar@eerhardt