xamarin-macios icon indicating copy to clipboard operation
xamarin-macios copied to clipboard

Nullability annotations on NSUrl

Open Therzok opened this issue 3 years ago • 36 comments

Therzok avatar Jun 27 '22 14:06 Therzok

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Build packages'

Pipeline on Agent Hash: [PR build]

@Therzok can you fix all the following warnings?

* Assertion at mono-threads.c:651, condition `info' not met, function:mono_thread_info_current, 
Foundation/NSUrlSessionHandler.cs(716,24): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(754,56): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrl.cs(58,21): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString, UriKind uriKind)'.
Foundation/NSUrl.cs(58,21): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString, UriKind uriKind)'.
Foundation/NSUrlSessionHandler.cs(716,24): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(754,56): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(716,24): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(754,56): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrl.cs(58,21): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString, UriKind uriKind)'.

@tj-devel709 give a hand if it is needed since you have experience with this changes.

mandel-macaque avatar Jun 27 '22 14:06 mandel-macaque

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Detect API changes'

Pipeline on Agent Hash: [PR build]

:fire: Unable to find the contents for the comment: D:\a\1\s\change-detection\results\gh-comment.md does not exist :fire

Pipeline on Agent Hash: 9f3e334aa1f3ce55c70f2eb6585fdfb4387cf945 [PR build]

@Therzok can you fix all the following warnings?

* Assertion at mono-threads.c:651, condition `info' not met, function:mono_thread_info_current, 
Foundation/NSUrlSessionHandler.cs(716,24): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(754,56): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrl.cs(58,21): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString, UriKind uriKind)'.
Foundation/NSUrl.cs(58,21): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString, UriKind uriKind)'.
Foundation/NSUrlSessionHandler.cs(716,24): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(754,56): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(716,24): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrlSessionHandler.cs(754,56): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString)'.
Foundation/NSUrl.cs(58,21): warning CS8604: Possible null reference argument for parameter 'uriString' in 'Uri.Uri(string uriString, UriKind uriKind)'.

@tj-devel709 give a hand if it is needed since you have experience with this changes.

Sure. I'm not sure about the assertion though, is it crashing on a null being passed?

Therzok avatar Jun 28 '22 19:06 Therzok

The assertion is probably csc triggering a Mono/CoreCLR bug (causing mono or coreclr to crash) - in other words not a problem with this PR, so you can ignore it.

rolfbjarne avatar Jun 28 '22 19:06 rolfbjarne

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Detect API changes'

Pipeline on Agent Hash: [PR build]

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Build packages'

Pipeline on Agent Hash: [PR build]

:fire: Unable to find the contents for the comment: D:\a\1\s\change-detection\results\gh-comment.md does not exist :fire

Pipeline on Agent Hash: 060ed97b8a0d0379f31e92e999a6ee38f9755857 [PR build]

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Build packages'

Pipeline on Agent Hash: [PR build]

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Detect API changes'

Pipeline on Agent Hash: [PR build]

:fire: Unable to find the contents for the comment: D:\a\1\s\change-detection\results\gh-comment.md does not exist :fire

Pipeline on Agent Hash: c0108ee6b2bf639858f71a0e3910fd7549415923 [PR build]

/azp run

mandel-macaque avatar Jun 30 '22 21:06 mandel-macaque

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jun 30 '22 21:06 azure-pipelines[bot]

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Detect API changes'

Pipeline on Agent Hash: [PR build]

:fire: Unable to find the contents for the comment: D:\a\1\s\change-detection\results\gh-comment.md does not exist :fire

Pipeline on Agent Hash: c0108ee6b2bf639858f71a0e3910fd7549415923 [PR build]

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Build packages'

Pipeline on Agent Hash: [PR build]

:computer: [PR Build] Tests on macOS Mac Catalina (10.15) passed :computer:

:white_check_mark: All tests on macOS Mac Catalina (10.15) passed.

Pipeline on Agent Hash: ac7312cd84ed5833683f0577922a315fd90e5649 [PR build]

:books: [PR Build] Artifacts :books:

Packages generated

View packages

Pipeline on Agent XAMBOT-1030.Monterey' Hash: ac7312cd84ed5833683f0577922a315fd90e5649 [PR build]

:white_check_mark: API diff for current PR / commit

Legacy Xamarin (No breaking changes)
  • ~iOS~ (no change detected)
  • ~tvOS~ (no change detected)
  • ~watchOS~ (no change detected)
  • ~macOS~ (no change detected)
NET (empty diffs)
  • ~iOS~: (empty diff detected)
  • ~tvOS~: (empty diff detected)
  • ~MacCatalyst~: (empty diff detected)
  • ~macOS~: (empty diff detected)

:white_check_mark: API diff vs stable

Legacy Xamarin (No breaking changes)
.NET (No breaking changes)
Legacy Xamarin (stable) vs .NET

:white_check_mark: Generator diff

Generator diff is empty

Pipeline on Agent Hash: ac7312cd84ed5833683f0577922a315fd90e5649 [PR build]

:x: [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) failed :x:

Failed tests are:

  • xammac_tests
  • monotouch-test

Pipeline on Agent Hash: ac7312cd84ed5833683f0577922a315fd90e5649 [PR build]

:fire: [CI Build] Test results :fire:

Test results

:x: Tests failed on VSTS: simulator tests

0 tests crashed, 7 tests failed, 216 tests passed.

Failures

:x: monotouch tests

4 tests failed, 19 tests passed.
  • monotouch-test/Mac Catalyst [dotnet]/Debug [dotnet]: Failed (Test run failed. Tests run: 2758 Passed: 2647 Inconclusive: 9 Failed: 1 Ignored: 110)
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (LinkSdk): Failed
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (static registrar): Failed
  • monotouch-test/iOS Unified 64-bits - simulator/Release (all optimizations): Failed

Html Report (VSDrops) Download

:x: mtouch tests

1 tests failed, 0 tests passed.
  • MTouch tests/NUnit: Failed (Execution failed with exit code 19)

Html Report (VSDrops) Download

:x: xtro tests

2 tests failed, 0 tests passed.
  • Xtro/Legacy Xamarin: BuildFailure
  • Xtro/.NET: BuildFailure

Html Report (VSDrops) Download

Successes

:white_check_mark: bcl: All 69 tests passed. Html Report (VSDrops) Download :white_check_mark: cecil: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: fsharp: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: framework: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: generator: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: interdependent_binding_projects: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: install_source: All 1 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 12 tests passed. Html Report (VSDrops) Download :white_check_mark: msbuild: All 2 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

Pipeline on Agent Hash: [PR build]

:fire: [PR Build] Build failed :fire:

Build failed for the job 'Detect API changes'

Pipeline on Agent Hash: [PR build]

:fire: Unable to find the contents for the comment: D:\a\1\s\change-detection\results\gh-comment.md does not exist :fire

Pipeline on Agent Hash: c0c4d5cbe6ce87e0a99f7651ee4449f2faef17cd [PR build]

:computer: [PR Build] Tests on macOS Mac Catalina (10.15) passed :computer:

:white_check_mark: All tests on macOS Mac Catalina (10.15) passed.

Pipeline on Agent Hash: c0c4d5cbe6ce87e0a99f7651ee4449f2faef17cd [PR build]

:fire: Failed to compare API and create generator diff :fire:

Error: 'make' failed for the hash 35654a1905985ffdcdfc44df828915b3753209c9.

Pipeline on Agent Hash: c0c4d5cbe6ce87e0a99f7651ee4449f2faef17cd [PR build]

:books: [PR Build] Artifacts :books:

Packages generated

View packages

Pipeline on Agent XAMBOT-1042.Monterey' Hash: c0c4d5cbe6ce87e0a99f7651ee4449f2faef17cd [PR build]

:x: [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) failed :x:

Failed tests are:

  • xammac_tests
  • monotouch-test

Pipeline on Agent Hash: c0c4d5cbe6ce87e0a99f7651ee4449f2faef17cd [PR build]

:fire: Failed to compare API and create generator diff :fire:

Error: 'make' failed for the hash 35654a1905985ffdcdfc44df828915b3753209c9.

Pipeline on Agent Hash: c0c4d5cbe6ce87e0a99f7651ee4449f2faef17cd [PR build]

:fire: [CI Build] Test results :fire:

Test results

:x: Tests failed on VSTS: simulator tests

0 tests crashed, 6 tests failed, 217 tests passed.

Failures

:x: monotouch tests

3 tests failed, 20 tests passed.
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (LinkSdk): Failed
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (static registrar): Failed
  • monotouch-test/iOS Unified 64-bits - simulator/Release (all optimizations): Failed

Html Report (VSDrops) Download

:x: mtouch tests

1 tests failed, 0 tests passed.
  • MTouch tests/NUnit: Failed (Execution failed with exit code 19)

Html Report (VSDrops) Download

:x: xtro tests

2 tests failed, 0 tests passed.
  • Xtro/Legacy Xamarin: BuildFailure
  • Xtro/.NET: BuildFailure

Html Report (VSDrops) Download

Successes

:white_check_mark: bcl: All 69 tests passed. Html Report (VSDrops) Download :white_check_mark: cecil: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: dotnettests: All 1 tests passed. Html Report (VSDrops) Download :white_check_mark: fsharp: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: framework: All 8 tests passed. Html Report (VSDrops) Download :white_check_mark: generator: All 2 tests passed. Html Report (VSDrops) Download :white_check_mark: interdependent_binding_projects: All 7 tests passed. Html Report (VSDrops) Download :white_check_mark: install_source: All 1 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 12 tests passed. Html Report (VSDrops) Download :white_check_mark: msbuild: All 2 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

Pipeline on Agent Hash: [PR build]