Skip to content

Backport VMR patch for cross builds#99702

Merged
jkoritzinsky merged 2 commits intomainfrom
jkoritzinsky-patch-1
Mar 14, 2024
Merged

Backport VMR patch for cross builds#99702
jkoritzinsky merged 2 commits intomainfrom
jkoritzinsky-patch-1

Conversation

@jkoritzinsky
Copy link
Member

Backport the VMR patch that fixed our cross-build issues.

Backport the VMR patch that fixed our cross-build issues.
@dotnet-policy-service
Copy link
Contributor

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

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jkoritzinsky I just noticed that we already pass the -cross switch in:

<InnerBuildArgs Condition="'$(TargetArch)' != '$(_hostArch)' and '$(ShortStack)' != 'true'">$(InnerBuildArgs) $(FlagParameterPrefix)cross</InnerBuildArgs>

@jkoritzinsky
Copy link
Member Author

That condition isn't correct for the x64 cross build. I'll merge the two conditions.

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.

2 participants