Skip to content

Add -cross flag to build.ps1#100678

Merged
akoeplinger merged 1 commit intomainfrom
akoeplinger-patch-1
Apr 5, 2024
Merged

Add -cross flag to build.ps1#100678
akoeplinger merged 1 commit intomainfrom
akoeplinger-patch-1

Conversation

@akoeplinger
Copy link
Member

Fixes VMR build error on Windows after dotnet/installer#19321

We haven't used or passed this on Windows before and a bunch of logic is conditioned based on the CrossBuild property (which is what -cross is turned into in build.sh) so ignore it for now.

Fixes VMR build error on Windows after dotnet/installer#19321

We haven't used or passed this on Windows before and a bunch of logic is conditioned based on the `CrossBuild` property (which is what `-cross` is turned into in build.sh) so ignore it for now.
@akoeplinger akoeplinger requested a review from ViktorHofer April 5, 2024 14:05
@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Apr 5, 2024
@teo-tsirpanis teo-tsirpanis added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Apr 5, 2024
@akoeplinger
Copy link
Member Author

/ba-g windows legs passed, the non-windows legs can't be affected.

@akoeplinger akoeplinger merged commit d79a753 into main Apr 5, 2024
@akoeplinger akoeplinger deleted the akoeplinger-patch-1 branch April 5, 2024 16:14
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
Fixes VMR build error on Windows after dotnet/installer#19321

We haven't used or passed this on Windows before and a bunch of logic is conditioned based on the `CrossBuild` property (which is what `-cross` is turned into in build.sh) so ignore it for now.
@github-actions github-actions bot locked and limited conversation to collaborators May 6, 2024
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.

3 participants