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

[Metal] Add support for xcode 13 beta5 . #12875

Merged
merged 5 commits into from
Oct 5, 2021

Conversation

mandel-macaque
Copy link
Member

No description provided.

@mandel-macaque mandel-macaque added the note-highlight Worth calling out specifically in release notes label Sep 29, 2021
@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

15 tests failed, 87 tests passed.

Failed tests

  • introspection/Mac Catalyst [dotnet]/Debug [dotnet]: BuildFailure
  • introspection/iOS Unified 64-bits - simulator/Debug: BuildFailure
  • introspection/tvOS - simulator/Debug: BuildFailure
  • introspection/watchOS 32-bits - simulator/Debug: BuildFailure
  • introspection/iOS Unified 64-bits - simulator/Debug (iOS 11.4): BuildFailure
  • introspection/tvOS - simulator/Debug (tvOS 11.4): BuildFailure
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 6.0): BuildFailure
  • monotouch-test/tvOS - simulator/Debug: BuildFailure
  • monotouch-test/tvOS - simulator/Debug (LinkSdk): BuildFailure
  • monotouch-test/tvOS - simulator/Debug (static registrar): BuildFailure
  • monotouch-test/tvOS - simulator/Release (all optimizations): BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug: BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug (LinkSdk): BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Release (all optimizations): BuildFailure

Pipeline on Agent XAMBOT-1101.BigSur'
Merge 7bce2b8 into af41600

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

9 tests failed, 93 tests passed.

Failed tests

  • link sdk/Mac Catalyst [dotnet]/Release [dotnet]: Failed (Tests run: 129 Passed: 112 Inconclusive: 6 Failed: 1 Ignored: 16)
  • monotouch-test/tvOS - simulator/Debug: BuildFailure
  • monotouch-test/tvOS - simulator/Debug (LinkSdk): BuildFailure
  • monotouch-test/tvOS - simulator/Debug (static registrar): BuildFailure
  • monotouch-test/tvOS - simulator/Release (all optimizations): BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug: BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug (LinkSdk): BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): BuildFailure
  • monotouch-test/watchOS 32-bits - simulator/Release (all optimizations): BuildFailure

Pipeline on Agent XAMBOT-1104.BigSur'
Merge f30fdb7 into af41600

src/Metal/MTLDevice.cs Show resolved Hide resolved
tests/introspection/iOS/iOSApiSelectorTest.cs Show resolved Hide resolved
src/metal.cs Show resolved Hide resolved
src/metal.cs Outdated Show resolved Hide resolved
src/metal.cs Outdated Show resolved Hide resolved
src/metal.cs Outdated Show resolved Hide resolved
src/metal.cs Outdated Show resolved Hide resolved
Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 Tests failed catastrophically on Build (no summary found). 🔥

Result file $(TEST_SUMMARY_PATH) not found.

Pipeline on Agent
Merge daa50ed into 3ffabb1

[Abstract, Export ("didModifyRange:")]
void DidModify (NSRange range);
#endif
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: extra tab

[Export ("vertical")]
MTLRasterizationRateSampleArray Vertical { get; }
*/

[Mac (12, 0), iOS (15, 0), TV (15,0), MacCatalyst(15,0), NoWatch]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Mac (12, 0), iOS (15, 0), TV (15,0), MacCatalyst(15,0), NoWatch]
[Mac (12, 0), iOS (15, 0), TV (15,0), MacCatalyst (15,0), NoWatch]

nit: spacing

[Export ("maxSampleCount")]
MTLSize MaxSampleCount { get; }

[Mac (12, 0), iOS (15, 0), TV (15,0), MacCatalyst(15,0), NoWatch]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[Mac (12, 0), iOS (15, 0), TV (15,0), MacCatalyst(15,0), NoWatch]
[Mac (12, 0), iOS (15, 0), TV (15,0), MacCatalyst (15,0), NoWatch]

nit: spacing

@@ -5243,4 +5563,152 @@ interface MTLFunctionLogDebugLocation {
nuint Column { get; }
}

[Mac (12,0), iOS (15,0), TV (15,0), MacCatalyst (15, 0), NoWatch]
[BaseType (typeof (NSObject))]
interface MTLStitchedLibraryDescriptor : NSCopying
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should all these "MTL..." classes be "Mtl..." ?

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

5 tests failed, 97 tests passed.

Failed tests

  • monotouch-test/Mac Catalyst [dotnet]/Debug [dotnet]: BuildFailure
  • monotouch-test/tvOS - simulator/Debug: BuildFailure
  • monotouch-test/tvOS - simulator/Debug (LinkSdk): BuildFailure
  • monotouch-test/tvOS - simulator/Debug (static registrar): BuildFailure
  • monotouch-test/tvOS - simulator/Release (all optimizations): BuildFailure

Pipeline on Agent XAMBOT-1104.BigSur'
Merge ba86323 into 3ffabb1

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

1 tests failed, 101 tests passed.

Failed tests

  • link sdk/Mac Catalyst [dotnet]/Debug [dotnet]: TimedOut (Execution timed out after 1200 seconds.
    No test log file was produced)

Pipeline on Agent XAMBOT-1094.BigSur'
Merge 9ff5c7a into 05ad66d

@mandel-macaque mandel-macaque merged commit 67e1ca5 into xamarin:main Oct 5, 2021
@mandel-macaque mandel-macaque deleted the metal-xcode13-beta5 branch October 5, 2021 17:32
@mandel-macaque
Copy link
Member Author

/sudo backport xcode13-ios

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Backport Job to branch xcode13-ios Created! The magic is happening here

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Oh no! Backport failed! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5293493 for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
note-highlight Worth calling out specifically in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants