-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Convert dotnet/runtime to use the new Arcade tooling for RPMs #109526
Conversation
… linux non-musl, including in the VMR. Fixes dotnet/source-build#4138
…or producing them in separate steps in special containers and remove our references to those containers
|
||
<PropertyGroup> | ||
<BuildDebPackage Condition="'$(TargetsLinuxGlibc)' == 'true'">true</BuildDebPackage> | ||
<BuildRpmPackage Condition="'$(TargetsLinuxGlibc)' == 'true'">true</BuildRpmPackage> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are set in .proj files, do we also need them in .props file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. This provides the defaults for the installers, but the runtime-deps installers only build some of the flavors (deb or rpm, not both).
Looks like cpio isn't available in the Azl3 docker images we use... I might have to update the image to install it. |
Apparently Docker image updates in dotnet/dotnet-buildtools-prereqs-docker#1247 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing, nice diff! Especially the YML part.
/ba-g unrelated work item crash |
Fixes #109518
Fixes dotnet/source-build#4138
Contributes to dotnet/source-build#3986
Depends on dotnet/arcade#15217