Skip to content

Revert back to winapi dependency for v0.4#340

Merged
Thomasdezeeuw merged 1 commit into
rust-lang:v0.4.xfrom
Thomasdezeeuw:revert_to_winapi
Aug 26, 2022
Merged

Revert back to winapi dependency for v0.4#340
Thomasdezeeuw merged 1 commit into
rust-lang:v0.4.xfrom
Thomasdezeeuw:revert_to_winapi

Conversation

@Thomasdezeeuw

Copy link
Copy Markdown
Collaborator

As noted in #308, switching
from winapi to windows-sys was a breaking change as we expose some of
the winapi types that are different in windows-sys.

This reverts the following commits:

  • "Relax windows-sys dependency version" 81056b9.
  • "Support Socket::(set_)recv_tos on Windows" 242e087.
  • "Replace winapi with windows-sys" aeb6c2d.

We'll make this switch in v0.5 instead.

As noted in <rust-lang#308>, switching
from winapi to windows-sys was a breaking change as we expose some of
the winapi types that are different in windows-sys.

This reverts the following commits:
 * "Relax windows-sys dependency version" 81056b9.
 * "Support Socket::(set_)recv_tos on Windows" 242e087.
 * "Replace winapi with windows-sys" aeb6c2d.

We'll make this switch in v0.5 instead.
@Thomasdezeeuw Thomasdezeeuw requested a review from Darksonn August 26, 2022 11:21
@Thomasdezeeuw Thomasdezeeuw changed the title Revert back to winapi dependency Revert back to winapi dependency for v0.4 Aug 26, 2022
@Thomasdezeeuw Thomasdezeeuw mentioned this pull request Aug 26, 2022

@Darksonn Darksonn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, but wasn't this already reverted?

@Thomasdezeeuw

Copy link
Copy Markdown
Collaborator Author

LGTM, but wasn't this already reverted?

No, we just yanked the v0.4.5 release.

@Thomasdezeeuw Thomasdezeeuw merged commit daf64ce into rust-lang:v0.4.x Aug 26, 2022
@Thomasdezeeuw Thomasdezeeuw deleted the revert_to_winapi branch August 26, 2022 12:32
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