Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Aug 30, 2021

Backport of #56426 to release/6.0

/cc @akoeplinger @uweigand

Customer Impact

Make isMSBuildOnNETCoreSupported return true on s390x, this fixes generating version headers when building on s390x.

Adds s390x platform-specific library name in find_unwind_libs, this enables linking against system-provided libunwind on s390x.

Testing

Manual testing.

Risk

Very low, we don't build s390x in our official builds.

* Make isMSBuildOnNETCoreSupported return true on s390x
  (fixes generating version headers)

* Add s390x platform-specific library name in find_unwind_libs
  (enables linking against system-provided libunwind)
@ghost
Copy link

ghost commented Aug 30, 2021

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.

@ghost
Copy link

ghost commented Aug 30, 2021

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

Issue Details

Backport of #56426 to release/6.0

/cc @akoeplinger @uweigand

Customer Impact

Make isMSBuildOnNETCoreSupported return true on s390x, this fixes generating version headers when building on s390x.

Adds s390x platform-specific library name in find_unwind_libs, this enables linking against system-provided libunwind on s390x.

Testing

Manual testing.

Risk

Very low, we don't build s390x in our official builds.

Author: github-actions[bot]
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@akoeplinger akoeplinger merged commit 5a17bd8 into release/6.0 Aug 31, 2021
@akoeplinger akoeplinger deleted the backport/pr-56426-to-release/6.0 branch August 31, 2021 10:30
@ghost ghost locked as resolved and limited conversation to collaborators Sep 30, 2021
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.

5 participants