Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[d17-2] [introspection] Fix according to macOS 12.4. Fixes #15229. #15512

Conversation

vs-mobiletools-engineering-service2
Copy link
Collaborator

  • Ignore anything Chip-related, the framework was removed from macOS.
  • Fix reporting errors some protocol-related errors so that they're actually
    reported as errors (by calling 'ReportError' instead of just adding the
    errors to a list that doesn't affect anything else).
  • Ignore the NSUserActivityRestoring protocol, according to the macOS headers
    it's attached to NSResponder using a category, which is invisible at
    runtime. Due to the previous point this didn't show up before, but once at
    least another error (which happened with the Chip framework), then this was
    listed as well.

Fixes #15229.

Backport of #15230

* Ignore anything Chip-related, the framework was removed from macOS.
* Fix reporting errors some protocol-related errors so that they're actually
  reported as errors (by calling 'ReportError' instead of just adding the
  errors to a list that doesn't affect anything else).
* Ignore the NSUserActivityRestoring protocol, according to the macOS headers
  it's attached to NSResponder using a category, which is invisible at
  runtime. Due to the previous point this didn't show up before, but once at
  least another error (which happened with the Chip framework), then this was
  listed as well.

Fixes xamarin#15229.
@vs-mobiletools-engineering-service2
Copy link
Collaborator Author

❌ Tests on macOS M1 - Mac Big Sur (11.5) failed ❌

No test package could be found for tests on macOS M1 - Mac Big Sur (11.5)

Pipeline on Agent
Merge ac70f98 into 10f3957

@vs-mobiletools-engineering-service2
Copy link
Collaborator Author

❌ Tests on macOS Mac Catalina (10.15) failed ❌

No test package could be found for tests on macOS Mac Catalina (10.15)

Pipeline on Agent
Merge ac70f98 into 10f3957

@vs-mobiletools-engineering-service2
Copy link
Collaborator Author

❌ [CI Build] Tests failed on VSTS: simulator tests iOS ❌

Tests failed on VSTS: simulator tests iOS.

Test results

6 tests failed, 99 tests passed.

Failed tests

  • introspection/Mac [dotnet]/Debug [dotnet]: BuildFailure
  • introspection/Mac Catalyst [dotnet]/Debug [dotnet]: BuildFailure
  • introspection/Mac Modern/Debug: BuildFailure
  • introspection/iOS Unified 64-bits - simulator/Debug: BuildFailure
  • introspection/tvOS - simulator/Debug: BuildFailure
  • introspection/watchOS 32-bits - simulator/Debug: BuildFailure

Pipeline on Agent XAMBOT-1161.Monterey'
Merge ac70f98 into 10f3957

@rolfbjarne
Copy link
Member

Closing, since this is outdated now.

@rolfbjarne rolfbjarne closed this Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported not-notes-worthy Ignore for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants