Skip to content

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

Closed
vs-mobiletools-engineering-service2 wants to merge 2 commits into
dotnet:d17-2from
vs-mobiletools-engineering-service2:backport-pr-15230-to-d17-2
Closed

vs-mobiletools-engineering-service2 wants to merge 2 commits into
dotnet:d17-2from
vs-mobiletools-engineering-service2:backport-pr-15230-to-d17-2

Conversation

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
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 dotnet#15229.
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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