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

[VMR] Light up Alpine lanes #19332

Merged
merged 7 commits into from
Apr 10, 2024
Merged

[VMR] Light up Alpine lanes #19332

merged 7 commits into from
Apr 10, 2024

Conversation

directhex
Copy link

@directhex directhex requested review from a team as code owners April 5, 2024 20:28
@dotnet dotnet deleted a comment from azure-pipelines bot Apr 5, 2024
@richlander
Copy link
Member

richlander commented Apr 5, 2024

This is exciting to see, adding first class support for Alpine in our new build.

JFYI: @ncopa @ayakael

@dotnet dotnet deleted a comment from azure-pipelines bot Apr 9, 2024
@ViktorHofer
Copy link
Member

Please wait until #19366 is merged and the merge with latest main to make sure that the tests work with these added lanes.

@directhex
Copy link
Author

/azp run installer-unified-build-full

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dotnet dotnet deleted a comment from azure-pipelines bot Apr 10, 2024
@dotnet dotnet deleted a comment from azure-pipelines bot Apr 10, 2024
@directhex
Copy link
Author

windowsdesktop failure is unrelated.

@directhex directhex merged commit 88a7cec into dotnet:main Apr 10, 2024
67 of 73 checks passed
useMonoRuntime: true
targetOS: linux-musl
targetArchitecture: x64
extraProperties: /p:TargetRID=linux-musl-x64
Copy link
Member

Choose a reason for hiding this comment

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

Why do we need this?

Copy link
Member

Choose a reason for hiding this comment

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

I just noticed that the TargetRID isn't calculated based on TargetOS and TargetArchitecture which seems weird to me but is entirely unrelated to this change.

@ayakael
Copy link

ayakael commented Apr 13, 2024

Happy to see this! I've divested from developping the Alpine dotnet package further due to lack of time, but hopefully this will help get better support for mono which is what cripples mono-based runtimes on musl.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants