microsoft / cppwinrt

C++/WinRT
MIT License
1.64k stars 238 forks source link

Allow classic COM interfaces with get_self #1314

Closed sylveon closed 1 year ago

sylveon commented 1 year ago

Fixes #1312

github-actions[bot] commented 1 year ago

This pull request is stale because it has been open 10 days with no activity. Remove stale label or comment or this will be closed in 5 days.

sylveon commented 1 year ago

@DefaultRyan can you take a look at this whenever convenient?

kennykerr commented 1 year ago

The good news is that the OS recently ditched a very very old version of cppwinrt that was only used by Xaml so now there is only one version of cppwinrt in the OS. The bad news is that one newer version is still over a year old, and the later versions introduce build breaks that have thus far prevented updating the OS. Unfortunately, until that is cleared up it won't be easy to update cppwinrt without risking additional delays. Ryan is looking into updating the OS but it may be some time before that's cleared up.

sylveon commented 1 year ago

That sounds like a urge ordeal! Hopefully these get resolved soon. In the meantime, is it possible to prevent the stale bot from closing PRs?

kennykerr commented 1 year ago

I assume you can always reopen the PRs later on if they are closed. And if you just keep commenting on the issue or PR from time to time then it will remain open.

github-actions[bot] commented 1 year ago

This pull request is stale because it has been open 10 days with no activity. Remove stale label or comment or this will be closed in 5 days.

sylveon commented 1 year ago

Bump, waiting on @DefaultRyan's review.

kennykerr commented 1 year ago

Unfortunately, a project maintainer is not currently available to review this pull request. Please see the contributing guide for more information. Feel free to keep the conversation going on the related issue.

jonwis commented 1 year ago

Reopening, will review Monday. Looks great so far.

kennykerr commented 1 year ago

@jonwis - this change is library-only so its relatively simple to review provided good test coverage.

sylveon commented 1 year ago

Any chance at a new release with this fix?

kennykerr commented 1 year ago

Kicked off a build.

kennykerr commented 1 year ago

Here you go: https://github.com/microsoft/cppwinrt/releases/tag/2.0.230706.1