Skip to content

Add tcp retries to windows #557

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

Merged

Conversation

keithmattix
Copy link
Contributor

Bump windows sys and add support for retries on Windows

Cargo.toml Outdated
@@ -54,7 +55,7 @@ features = ["all"]
libc = "0.2.150"

[target.'cfg(windows)'.dependencies.windows-sys]
version = "0.52"
version = "0.59"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a breaking change, can't accept that at the moment, see #526

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you need a volunteer to implement #526 (comment)? I'm happy to take a stab at it, or pass the feedback to someone internally at MSFT

Copy link
Contributor

Choose a reason for hiding this comment

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

windows-sys was updated to 0.59 in #579.

@Thomasdezeeuw Thomasdezeeuw force-pushed the add-tcp-retries-to-windows branch from 3b6ea10 to 1ea4091 Compare May 25, 2025 11:16
@Thomasdezeeuw Thomasdezeeuw force-pushed the add-tcp-retries-to-windows branch from 1ea4091 to 07f0700 Compare May 25, 2025 11:20
@Thomasdezeeuw Thomasdezeeuw merged commit 4c1cf6b into rust-lang:master May 25, 2025
44 of 46 checks passed
@Thomasdezeeuw
Copy link
Collaborator

Thanks @keithmattix

@keithmattix keithmattix deleted the add-tcp-retries-to-windows branch May 25, 2025 12:43
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.

3 participants