[main] Source code updates from dotnet/runtime#4727
Conversation
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
|
@steveisok the build with ninja by default und Linux and Mac needs a reaction here. Looks like the images don't have Ninja installed. |
|
|
What needs to be done for Mac? |
Is there a failure on the mac I'm not seeing? |
|
The iossimulator job which uses macos is passing so apparently not. Does that job provide enough validation? |
It'll check for ninja and fail, so yes. |
|
I meant if we think that's sufficient given that we use another machine pool (arm64) internally. But we can find out post-merge. |
Most likely caused by dotnet/runtime#124077 but I don't understand why this even gets raised because the warning is suppressed in code: |
|
oh I see, the PR has a notice:
|
Note
This is a codeflow update. It may contain both source code changes from
the source repo
as well as dependency updates. Learn more here.
This pull request brings the following source code changes
From https://github.com/dotnet/runtime
Diff the source with this PR branch