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

Fix source-build on the s390x platform #58553

Merged
merged 1 commit into from
Sep 3, 2021
Merged

Fix source-build on the s390x platform #58553

merged 1 commit into from
Sep 3, 2021

Conversation

uweigand
Copy link
Contributor

@uweigand uweigand commented Sep 2, 2021

  • Do not disable Mono in source-build if it is the primary runtime

This fixes a build failure when building runtime via the source-build method on s390x: with source-build, the Mono runtime is currently not supported -- but this doesn't make sense on s390x where Mono is the primary (only) supported runtime.

* Do not disable Mono in source-build if it is the primary runtime
@dotnet-issue-labeler
Copy link

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 ghost added the community-contribution Indicates that the PR has been added by a community member label Sep 2, 2021
@uweigand
Copy link
Contributor Author

uweigand commented Sep 2, 2021

CC @crummel @akoeplinger

@ViktorHofer
Copy link
Member

I'm probably not the right person to review this as it's runtime and mono specific. @akoeplinger can you please take a look?

@akoeplinger akoeplinger merged commit 3fa8c5c into dotnet:main Sep 3, 2021
@akoeplinger
Copy link
Member

/backport to release/6.0

@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2021

Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/1197682294

@uweigand uweigand deleted the s390x-sourcebuild branch September 21, 2021 09:27
@ghost ghost locked as resolved and limited conversation to collaborators Nov 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants