[net11.0] [dotnet] Remove workaround for dotnet/sdk#52480#25720
[net11.0] [dotnet] Remove workaround for dotnet/sdk#52480#25720rolfbjarne wants to merge 1 commit into
Conversation
The workaround overrode the _LinkReadyToRunMachO target to skip the built-in dylib creation. Now that dotnet/sdk#52480 has been merged, this is no longer needed. 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.
✅ [CI Build #d43652b] Build passed (Build packages) ✅Pipeline on Agent |
✅ [CI Build #d43652b] Build passed (Detect API changes) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ 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 |
✅ [CI Build #d43652b] Build passed (Build macOS tests) ✅Pipeline on Agent |
🔥 [CI Build #d43652b] Test results 🔥Test results❌ Tests failed on VSTS: test results 0 tests crashed, 17 tests failed, 178 tests passed. Failures❌ introspection tests1 tests failed, 8 tests passed.Failed tests
Html Report (VSDrops) Download ❌ linker tests (tvOS)11 tests failed, 0 tests passed.Failed tests
Html Report (VSDrops) Download ❌ xcframework tests2 tests failed, 2 tests passed.Failed tests
Html Report (VSDrops) Download ❌ Tests on macOS Sonoma (14) tests2 tests failed, 3 tests passed.Failed tests
Html Report (VSDrops) Download ❌ Tests on macOS Tahoe (26) tests1 tests failed, 4 tests passed.Failed tests
Html Report (VSDrops) Download Successes✅ cecil: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Sequoia (15): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
The
_LinkReadyToRunMachOtarget override was added as a temporary workaround to skip the built-in dylib creation until dotnet/sdk#52480 was merged.That PR has now been merged, so the workaround is no longer needed.
🤖 Pull request created by Copilot