Skip to content

Move Windows backend to winit-win32#4247

Merged
kchibisov merged 9 commits intomasterfrom
madsmtm/winit-win32
May 25, 2025
Merged

Move Windows backend to winit-win32#4247
kchibisov merged 9 commits intomasterfrom
madsmtm/winit-win32

Conversation

@madsmtm
Copy link
Copy Markdown
Member

@madsmtm madsmtm commented May 21, 2025

Part of #3433.

I'm choosing the name winit-win32 because that is the name of the actual API we're using, and because in the future it might make sense to have different Windows backends (such as winit-winrt). See #4246 for background.

I decided to not yet rename all the other user-facing things like extension traits etc., that can be done in a follow-up.

@madsmtm madsmtm requested a review from notgull as a code owner May 21, 2025 11:41
@madsmtm madsmtm added DS - win32 Affects the Win32/Windows backend S - maintenance Repaying technical debt labels May 21, 2025
@madsmtm madsmtm requested review from kchibisov and removed request for notgull May 21, 2025 11:41
@madsmtm madsmtm mentioned this pull request May 21, 2025
5 tasks
Copy link
Copy Markdown
Member

@kchibisov kchibisov left a comment

Choose a reason for hiding this comment

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

This looks fine, remember to reserve the names on crates.io for 0.0.0 which is just empty crate with winit's license.

@madsmtm
Copy link
Copy Markdown
Member Author

madsmtm commented May 21, 2025

This looks fine, remember to reserve the names on crates.io for 0.0.0 which is just empty crate with winit's license.

Didn't want to until we settled on the name. But yeah, I will.

@madsmtm madsmtm force-pushed the madsmtm/winit-win32 branch from 7a4e863 to 91b7237 Compare May 21, 2025 12:15
@kchibisov
Copy link
Copy Markdown
Member

Didn't want to until we settled on the name. But yeah, I will.

it's not like we can not rename later on tbf.

@madsmtm madsmtm requested a review from kchibisov May 21, 2025 17:30
@kchibisov kchibisov merged commit 3b986f5 into master May 25, 2025
57 checks passed
@kchibisov kchibisov deleted the madsmtm/winit-win32 branch May 25, 2025 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DS - win32 Affects the Win32/Windows backend S - maintenance Repaying technical debt

Development

Successfully merging this pull request may close these issues.

3 participants