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

eng/SourceBuild.props: remove dangling SourceBuildNonPortable. #82160

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

tmds
Copy link
Member

@tmds tmds commented Feb 15, 2023

Missed this in #81480.

@ViktorHofer ptal.

@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 Feb 15, 2023
@ViktorHofer ViktorHofer merged commit 3b7dfff into dotnet:main Feb 15, 2023
@oleksandr-didyk
Copy link
Contributor

@ViktorHofer @tmds Hey, ran into an issue related to this change on my Ubuntu 20.04 VM when source-building runtime.

When running ./build.sh --sb the build would fail on projects restore with the following exception:

error NETSDK1094: Unable to optimize assemblies for performance: a valid runtime package was not found.
Either set the PublishReadyToRun property to false, or use a supported runtime identifier when publishing and make sure to restore packages with the PublishReadyToRun property set to true.

When I reverted the change, it would finish successfully.

Inspecting the binlogs of the two builds, it seems that RuntimeOS and AdditionalRuntimeIdentifierParent are the main differences. In the failing build they would be set to ubuntu.20.04, in the passing one - linux.

Would be great if someone could take a look. Thanks!

@ghost ghost locked as resolved and limited conversation to collaborators Mar 25, 2023
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.

3 participants