Fix to build mono on Windows with ninja by default as well

[akoeplinger]

This got broken by 8c2158f since we no longer passed /p:Ninja=true down to mono.proj from build.cmd/ps1

Fixes #50753

[ghost]

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

Issue Details

---

This got broken by 8c2158f since we no longer passed /p:Ninja=true down to mono.proj

Fixes #50753

Author:	akoeplinger
Assignees:	-
Labels:	area-Infrastructure-mono
Milestone:	-

[akoeplinger]

Looks like this was missed in the initial PR because it didn't touch any files in src/mono so CI didn't trigger mono legs.

@safern should we add eng/* as triggers to eng/pipelines/common/evaluate-default-paths.yml?

[safern]

Hmmm that should already be included:

https://github.com/dotnet/runtime/blob/main/eng/pipelines/common/evaluate-default-paths.yml#L28-L45

[safern]

I do see mono build running on that PR build:

https://dev.azure.com/dnceng/public/_build/results?buildId=1054503&view=logs&j=e4304a73-856f-5230-44ef-10471e0044a5

Also this is the output for evaluate paths for the mono subset:

******* Probing mono exclude paths *******
eng/Version.Details.xml
*.md
LICENSE.TXT
PATENTS.TXT
THIRD-PARTY-NOTICES.TXT
docs/*
src/installer/*
src/coreclr/*
src/libraries/*
src/tests/*
eng/pipelines/installer/*
eng/pipelines/coreclr/*
eng/pipelines/libraries/*

+ git diff -M -C -b --ignore-cr-at-eol --ignore-space-at-eol --exit-code --quiet 'HEAD^1' -- ':!eng/Version.Details.xml' ':!*.md' ':!LICENSE.TXT' ':!PATENTS.TXT' ':!THIRD-PARTY-NOTICES.TXT' ':!docs/*' ':!src/installer/*' ':!src/coreclr/*' ':!src/libraries/*' ':!src/tests/*' ':!eng/pipelines/installer/*' ':!eng/pipelines/coreclr/*' ':!eng/pipelines/libraries/*'

----- Matching files for mono -----
+ git diff -M -C -b --ignore-cr-at-eol --ignore-space-at-eol --name-only 'HEAD^1' -- ':!eng/Version.Details.xml' ':!*.md' ':!LICENSE.TXT' ':!PATENTS.TXT' ':!THIRD-PARTY-NOTICES.TXT' ':!docs/*' ':!src/installer/*' ':!src/coreclr/*' ':!src/libraries/*' ':!src/tests/*' ':!eng/pipelines/installer/*' ':!eng/pipelines/coreclr/*' ':!eng/pipelines/libraries/*'
eng/build.ps1
eng/native/configurecompiler.cmake
eng/native/functions.cmake
eng/pipelines/global-build.yml
src/native/corehost/CMakeLists.txt
src/native/corehost/build.cmd

Setting pipeline variable containsChange=true

And here is the build for Browser on Windows that ran on that PR:
https://dev.azure.com/dnceng/public/_build/results?buildId=1054506&view=logs&jobId=4ffd1c85-149a-52c0-00e1-0d9c3af850ba&j=4ffd1c85-149a-52c0-00e1-0d9c3af850ba&t=9fe9d5a4-3635-5eda-7e39-9e2c5e3efbab

[akoeplinger]

@safern hmm you're right, it did run in the PR. I was confused because GitHub doesn't show it in the PR result summary at the end of the PR (only lists 100 statuses there) but does show it on the Checks tab:

vs.

This PR should still fix the issue though.

[akoeplinger]

Mono crossaot Product Build windows x64 release passed, the other failures are infrastructure issues.