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

🍳 Upgrade to objc2 0.6 and objc2-metal 0.3 #261

Merged
merged 2 commits into from
Feb 3, 2025
Merged

🍳 Upgrade to objc2 0.6 and objc2-metal 0.3 #261

merged 2 commits into from
Feb 3, 2025

Conversation

MarijnS95
Copy link
Member

The MTLCreateSystemDefaultDevice() function finally has a high-level wrapper that constructs a Retained protocol object for us.

Note also that a new MTLAllocation interface was added to Metal. Even if that interface is optional, current objc2 design requires this trait to be part of the hierarchy (and the feature explicitly enabled) to get access to all "descendant" interfaces (MTLResource, which provides MTLBuffer and MTLTexture). There are upstream plans to see if this can be simplified: madsmtm/objc2#663.

@MarijnS95 MarijnS95 force-pushed the objc2-0.6 branch 2 times, most recently from 1f6478d to 017ee6f Compare January 24, 2025 16:43
@MarijnS95 MarijnS95 enabled auto-merge (squash) January 24, 2025 16:48
@MarijnS95 MarijnS95 disabled auto-merge January 27, 2025 15:43
@MarijnS95 MarijnS95 enabled auto-merge (squash) January 27, 2025 15:44
@MarijnS95
Copy link
Member Author

Oh - the MSRV check has been renamed to 1.71 to match the bump... and branch protection rules need to be updated. Should we remove the version from the title before we do that?

@MarijnS95 MarijnS95 changed the title Upgrade to objc2 0.6 and objc2-metal 0.3 🍳 Upgrade to objc2 0.6 and objc2-metal 0.3 Jan 31, 2025
@Jasper-Bekkers
Copy link
Member

Updated the rules, just needs a rebase it seems.

@Jasper-Bekkers
Copy link
Member

Rebase went fine but it looks like commits aren't signed.

The `MTLCreateSystemDefaultDevice()` function finally has a high-level
wrapper that constructs a `Retained` protocol object for us.

Note also that a new `MTLAllocation` interface was added to Metal.  Even
if that interface is _optional_, current `objc2` design requires this
trait to be part of the hierarchy (and the feature explicitly enabled)
to get access to all "descendant" interfaces (`MTLResource`, which
provides `MTLBuffer` and `MTLTexture`).
@MarijnS95
Copy link
Member Author

Hmm seems GitHub no longer signs commits with its own key on rebase (for the better, probably).

@MarijnS95 MarijnS95 merged commit ba9a268 into main Feb 3, 2025
26 checks passed
@MarijnS95 MarijnS95 deleted the objc2-0.6 branch February 3, 2025 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants