Skip to content

Fix source-build on the s390x platform - #58553

Merged
akoeplinger merged 1 commit into
dotnet:mainfrom
uweigand:s390x-sourcebuild
Sep 3, 2021
Merged

Fix source-build on the s390x platform#58553
akoeplinger merged 1 commit into
dotnet:mainfrom
uweigand:s390x-sourcebuild

Conversation

@uweigand

Copy link
Copy Markdown
Contributor
  • 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
@ghost

ghost commented Sep 2, 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 2, 2021
@uweigand

Copy link
Copy Markdown
ContributorAuthor

CC @crummel@akoeplinger

@ViktorHofer

Copy link
Copy Markdown
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:mainSep 3, 2021
@akoeplinger

Copy link
Copy Markdown
Member

/backport to release/6.0

@github-actions

Copy link
Copy Markdown
Contributor

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
@ghostghost locked as resolved and limited conversation to collaborators Nov 3, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community-contributionIndicates 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

@uweigand@ViktorHofer@akoeplinger@crummel