Skip to content

Conversation

whollins
Copy link
Contributor

Motivation

Update the windows-sys dependency to the latest version.

Solution

Minor change needed as the BOOLEAN type has been removed but a standard bool is
the replacement.

@whollins whollins force-pushed the bump-windows-sys branch 3 times, most recently from c0eda79 to ed407b2 Compare September 23, 2025 22:05
@whollins whollins changed the title Bump windows-sys to version 0.61 chore: Bump windows-sys to version 0.61 Sep 23, 2025
@whollins whollins marked this pull request as ready for review September 23, 2025 22:53
@ADD-SP
Copy link
Member

ADD-SP commented Sep 24, 2025

Partially duplicate with #7469. @taiki-e Could you take a look?

@ADD-SP ADD-SP added the A-tokio Area: The main tokio crate label Sep 24, 2025
@taiki-e
Copy link
Member

taiki-e commented Sep 29, 2025

We need to update windows-sys in mio and socket2, which are our dependencies. There's already an open PR for mio (tokio-rs/mio#1901), but not for socket2. Could you open one for it?

@Darksonn
Copy link
Contributor

Problem is the new MSRV of 1.71 in windows-sys.

@ADD-SP ADD-SP added S-blocked-on-msrv Status: need an MSRV bump to progress and removed S-blocked-on-msrv Status: need an MSRV bump to progress labels Sep 30, 2025
@a1phyr
Copy link
Contributor

a1phyr commented Oct 4, 2025

This could probably solved as proposed in tokio-rs/mio#1901, by allowing both 0.60 and 0.61 versions.

@Thomasdezeeuw
Copy link
Contributor

Are the range versions a good solution to this? I'm not sure if Mio or Socket2 will hit a breaking change in them.

@Darksonn
Copy link
Contributor

Darksonn commented Oct 6, 2025

I bumped the MSRV to 1.71 in #7658, so we can land this. Please rebase.

@taiki-e taiki-e mentioned this pull request Oct 6, 2025
4 tasks
@whollins
Copy link
Contributor Author

whollins commented Oct 6, 2025

I bumped the MSRV to 1.71 in #7658, so we can land this. Please rebase.

Done

@Darksonn Darksonn merged commit 9255d96 into tokio-rs:master Oct 7, 2025
89 checks passed
@Darksonn Darksonn mentioned this pull request Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tokio Area: The main tokio crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants