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

Change how we build the cross-OS DAC to support building in the VMR #111927

Merged
merged 11 commits into from
Feb 4, 2025

Conversation

jkoritzinsky
Copy link
Member

  • Don't build the cross-os DACs as part of the default subsets. Breaks in them are very rare nowadays and we do have CI protection for them
  • Look up the runtimes for the cross-os DAC packages by pulling them out of the runtime packs, not using a different special artifact name.
  • Build the cross-os DACs and pack them all in one subset. This makes enabling building these components easier in the VMR.

Depends on #111416 (as this also removes the signing steps in the cross-os DAC job so we can do everything in one invocation).

Contributes to dotnet/source-build#4631

…quite rare nowadays and we have CI protection against build failures. Remove the linux dac from the default build subsets to speed up the default build experience.
Download less and don't extract into the same folder our sources are in

Correctly handle extracting nugets manually vs restoring them

Extract with a one line powershell script instead

Fix artifacts lookup folder.

Fix the extraction root

Just use Version for now and we'll figure out stabilization later

Try extracting into a separate powershell script and not using Expand-Archive. Also, adjust the layout to match NuGet package layout if we're writing our own script
Copy link
Contributor

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

@jkoritzinsky
Copy link
Member Author

jkoritzinsky commented Feb 3, 2025

Official build link with signing enabled: https://dev.azure.com/dnceng/internal/_build/results?buildId=2634189&view=results

@hoyosjs hoyosjs merged commit dea928c into dotnet:main Feb 4, 2025
149 of 151 checks passed
@jkoritzinsky jkoritzinsky deleted the xos-dac branch February 4, 2025 08:13
grendello added a commit to grendello/runtime that referenced this pull request Feb 4, 2025
* main: (30 commits)
  JIT: Optimize bit-wise AND with a constant mask in combination with a left shift in a compare (dotnet#111979)
  Change how we build the cross-OS DAC to support building in the VMR (dotnet#111927)
  Add Windows Server 2025 to test configurations (dotnet#111938)
  [PERF] Move performance testing YAML from dotnet/runtime to dotnet/performance (dotnet#111454)
  arm64: Add support for Bitwise OR NOT & XOR NOT (dotnet#111893)
  JIT: Fix cross crossgen comparison failures (dotnet#112078)
  Bump `StyleCop.Analyzers` to `1.2.0-beta.556` (dotnet#111278)
  Remove `RequiresProcessIsolation` on InterfaceFolding tests (dotnet#112098)
  Use hardlinks in helixpublishwitharcade (dotnet#112091)
  Update breaking change rules regarding byref/objref fields. (dotnet#112087)
  [daccess] Do not use USE_DAC_TABLE_RVA on Apple platforms (dotnet#112076)
  use collection syntax in illink (dotnet#108458)
  Include PDB for all TfmRuntimeSpecificPackageFile (dotnet#111879)
  [main] Update dependencies from dotnet/emsdk (dotnet#111690)
  Enable Mono tests (dotnet#111981)
  Let the debugger knows DATAS is on (dotnet#107115)
  Tests ran counter (dotnet#111145)
  Some System.Decimal performance improvements (dotnet#99212)
  [mono][mini] Remove support for the Xamarin.iOS and Xamarin.Mac assemblies in the AOT compiler. (dotnet#108886)
  Remove one usage of `Unsafe.AsPointer`. (dotnet#112079)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants