Closed simonrozsival closed 2 months ago
@rolfbjarne do you think this change (if approved) should go into main
or the net9.0
branch?
Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)
Pipeline on Agent Hash: d44d2c47d428612b33c94119a74c39f738168f78 [PR build]
/azp run
Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)
Pipeline on Agent Hash: d44d2c47d428612b33c94119a74c39f738168f78 [PR build]
/azp run
Had to re-trigger, something funny was going on with VSTS.
Pipeline on Agent Hash: [PR build]
Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)
Pipeline on Agent Hash: d44d2c47d428612b33c94119a74c39f738168f78 [PR build]
Pipeline on Agent Hash: [PR build]
Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)
Pipeline on Agent Hash: f85b5833e1f389b06f204e50d68dcdd3408350d3 [PR build]
Pipeline on Agent Hash: [PR build]
:white_check_mark: All tests on macOS X64 - Mac Sonoma (14) passed.
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Ventura (13) passed.
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Big Sur (11) passed.
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Monterey (12) passed.
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
:x: Failed :x:
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
:x: Tests failed on VSTS: test results
0 tests crashed, 2 tests failed, 166 tests passed.
Html Report (VSDrops) Download
Html Report (VSDrops) Download
:white_check_mark: cecil: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: framework: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: fsharp: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: generator: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: install-source: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: interdependent-binding-projects: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: introspection: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: linker: All 65 tests passed. Html Report (VSDrops) Download :white_check_mark: mac-binding-project: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: mmp: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mononative: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (iOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (MacCatalyst): All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (macOS): All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (tvOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (watchOS): All 4 tests passed. Html Report (VSDrops) Download :white_check_mark: msbuild: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mtouch: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: xammac: All 3 tests passed. Html Report (VSDrops) Download :white_check_mark: xcframework: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: xtro: All 2 tests passed. Html Report (VSDrops) Download
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
:x: Failed :x:
Pipeline on Agent Hash: 401c079fadffa50d73b7ee84768eb624831d9283 [PR build]
The failing tests are relevant (even though they don't seem to block the PR?):
Xamarin.Tests.DotNetProjectTest.PublishAotWithTrimAnalyzerWarningBuild(MacCatalyst,"maccatalyst-arm64"): Warning count
Expected: 1
But was: 29
I originally created this PR with the net9.0
branch as the base and later I rebased it to main
when the CI wouldn't start. I think I should rebase back to net9.0
where there aren't any additional trimming warnings even when we don't suppress the warnings.
even though they don't seem to block the PR?
A temporary situation while we deal with CI changes.
I originally created this PR with the
net9.0
branch as the base and later I rebased it tomain
when the CI wouldn't start. I think I should rebase back tonet9.0
where there aren't any additional trimming warnings even when we don't suppress the warnings.
Ah yes, that makes sense.
@jonathanpeppers
Considering these changes and to have the same behaviour across all platforms I think we should change the condition: https://github.com/dotnet/android/blob/4127db7e5514d1f90ecc33b1e07c0afccc837c87/src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.DefaultProperties.targets#L88
We will discuss how the condition needs to look like internally and the reasons why it need to be applied.
@ivanpovazan we have an issue listing all the NativeAOT TODOs, I linked to this PR:
Pipeline on Agent Hash: [PR build]
:white_check_mark: All tests on macOS X64 - Mac Sonoma (14) passed.
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:x: Failed :x:
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Ventura (13) passed.
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Monterey (12) passed.
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
Pipeline on Agent Hash: [PR build]
@ivanpovazan we have an issue listing all the NativeAOT TODOs, I linked to this PR:
Do you want us to create an issue about considering IsAotCompatible
in the Android SDK when SuppressTrimAnalysisWarnings
is being set?
:x: Tests failed on VSTS: test results
0 tests crashed, 3 tests failed, 167 tests passed.
Html Report (VSDrops) Download
Html Report (VSDrops) Download
Html Report (VSDrops) Download
:white_check_mark: cecil: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: framework: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: fsharp: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: generator: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: install-source: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: interdependent-binding-projects: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: introspection: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: linker: All 65 tests passed. Html Report (VSDrops) Download :white_check_mark: mac-binding-project: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: mmp: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mononative: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (iOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (macOS): All 9 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (tvOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: msbuild: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mtouch: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: xammac: All 3 tests passed. Html Report (VSDrops) Download :white_check_mark: xcframework: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: xtro: All 2 tests passed. Html Report (VSDrops) Download
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:white_check_mark: All tests on macOS X64 - Mac Sonoma (14) passed.
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Ventura (13) passed.
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:x: Failed :x:
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Monterey (12) passed.
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
:x: Tests failed on VSTS: test results
0 tests crashed, 1 tests failed, 169 tests passed.
Html Report (VSDrops) Download
:white_check_mark: cecil: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: framework: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: fsharp: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: generator: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: install-source: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: interdependent-binding-projects: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: introspection: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: linker: All 65 tests passed. Html Report (VSDrops) Download :white_check_mark: mac-binding-project: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: mmp: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mononative: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (iOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (MacCatalyst): All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (macOS): All 9 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (tvOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (watchOS): All 4 tests passed. Html Report (VSDrops) Download :white_check_mark: msbuild: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mtouch: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: xammac: All 3 tests passed. Html Report (VSDrops) Download :white_check_mark: xcframework: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: xtro: All 2 tests passed. Html Report (VSDrops) Download
Pipeline on Agent Hash: 4502d352c084dea963ea06f0bb49ce3479360678 [PR build]
Btw the test now fails with:
Xamarin.Tests.DotNetProjectTest.BuildMyNativeAotAppWithTrimAnalysisWarning(iOS,"iossimulator-arm64"): Warning count
Expected: 1
But was: 2
Build failed for the job 'Build packages'
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
Build failed for the job 'Build macOS tests'
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
Pipeline on Agent Hash: [PR build]
:white_check_mark: All tests on macOS X64 - Mac Sonoma (14) passed.
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Ventura (13) passed.
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
:white_check_mark: All tests on macOS M1 - Mac Monterey (12) passed.
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
:x: Failed :x:
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
:white_check_mark: All tests passed on VSTS: test results.
:tada: All 170 tests passed :tada:
:white_check_mark: cecil: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (iOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: framework: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: fsharp: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: generator: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: install-source: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: interdependent-binding-projects: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: introspection: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: linker: All 65 tests passed. Html Report (VSDrops) Download :white_check_mark: mac-binding-project: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: mmp: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mononative: All 6 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (iOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (MacCatalyst): All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (macOS): All 9 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (tvOS): All 11 tests passed. Html Report (VSDrops) Download :white_check_mark: monotouch (watchOS): All 4 tests passed. Html Report (VSDrops) Download :white_check_mark: msbuild: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: mtouch: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: xammac: All 3 tests passed. Html Report (VSDrops) Download :white_check_mark: xcframework: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: xtro: All 2 tests passed. Html Report (VSDrops) Download
Pipeline on Agent Hash: d98b5ff270d362cd7116fc6dae4b61dbf3ceafe8 [PR build]
@rolfbjarne with the changes in the last comment, the new tests are now passing (finally). There are just failing Windows tests but I saw this on other PRs and the error message hints that this is an infra issue and not related to this PR:
error VSX1002: The .NET runtime 8.0.1 or newer could not be found in the remote host under '/Users/builder/Library/Caches/Xamarin/XMA/SDKs/dotnet/dotnet'. Please install it or try setting a different path using the MSBuild property "DotNetRuntimePath".
Pipeline on Agent Hash: [PR build]
:white_check_mark: All tests on macOS X64 - Mac Sonoma (14) passed.
Pipeline on Agent Hash: 190e0b53394ab28a4884408de42d358b2a9c3c92 [PR build]
We noticed we weren't seeing trim analysis warnings in VS Code when PublishAot was set to true. There was a recent change that correctly disabled the suppressions when TrimMode is full. We need to make sure that we're also getting the trim analysis warnings in dotnet build with PublishAot but suppress them when publishing (in that case the warnings will come later from ILC). This PR aligns the behavior of PublishAot=true and TrimMode=true in debug builds.
/cc @ivanpovazan @rolfbjarne @jonathanpeppers