[mobile] .NET 11 mobile runtime configuration updates#25672
Conversation
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
There was a problem hiding this comment.
Pull request overview
This PR updates the mobile/runtime configuration and CI surface area to align with removing Mono mobile workloads/RIDs in the .NET 11 toolchain, primarily by disabling Mono-based test variations and stopping Mono runtime-pack downloads in the build/package-download step.
Changes:
- Disable Mono-based XHarness/Jenkins test variations by forcing
supports_mono=falseand gating interpreter/LLVM variations accordingly. - Remove Mono runtime usage from multiple .NET unit tests (Windows remote tests, build/package/linking tests, incremental build tests).
- Stop downloading Mono runtime packs in
builds/package-downloadand remove the corresponding Makefile property plumbing; add temporary SDK/feed pins for validation.
Show a summary per file
| File | Description |
|---|---|
| tests/xharness/Jenkins/TestVariationsFactory.cs | Forces Mono test variations off and gates interpreter/LLVM test variants behind supports_mono. |
| tests/dotnet/UnitTests/WindowsTest.cs | Removes Mono-specific remote Windows test cases (CoreCLR-only remaining). |
| tests/dotnet/UnitTests/ProjectTest.cs | Removes Mono-targeted test cases and Mono-only linked-framework expectations. |
| tests/dotnet/UnitTests/PostBuildTest.cs | Removes Mono variants for IPA/package publish tests. |
| tests/dotnet/UnitTests/IncrementalBuildTest.cs | Removes Mono-only incremental build tests and interpreter toggling tests. |
| tests/dotnet/UnitTests/BundleStructureTest.cs | Removes Mono build variants of bundle structure verification tests. |
| NuGet.config | Adds the general-testing feed (noted in PR description as a temporary validation pin). |
| global.json | Pins the repo SDK/tooling to a .NET 11 preview.6 build for validation. |
| eng/Version.Details.xml | Updates dotnet/dotnet dependency versions and SHAs to the pinned preview.6 build. |
| builds/package-download/download-packages.csproj | Removes Mono runtime pack download items (CoreCLR runtime packs only). |
| builds/Makefile | Stops passing MonoVM runtime identifier property into package-download build. |
Copilot's findings
- Files reviewed: 11/11 changed files
- Comments generated: 3
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
rolfbjarne
left a comment
rolfbjarne
left a comment
There was a problem hiding this comment.
Also there's a merge conflict in the PR now.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove the .NET unit tests that build iOS, tvOS and MacCatalyst apps with UseMonoRuntime=true. These resolve the Mono mobile runtime packs, which are no longer produced. macOS Mono cases are kept. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@kotlarmilos FYI I changed the PR a little bit, we now build/include/ship Mono bits, but all the tests are still disabled. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ [PR Build #46d7a59] Build passed (Detect API changes) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
✅ [PR Build #46d7a59] Build passed (Build packages) ✅Pipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
✅ [PR Build #46d7a59] Build passed (Build macOS tests) ✅Pipeline on Agent |
🚀 [CI Build #46d7a59] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 228 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Sonoma (14): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
Add two variables to determine how Mono is handled:
DOTNET_MONOVM_SUPPORTED: if Mono is still supported (in which case we'll run tests using Mono).DOTNET_MONOVM_STILL_PRESENT: whether we still include Mono bits in the build and ship those bits.The former is disabled, but the latter is still enabled for emergencies.