Skip to content
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

bump windows-sys dependency to 0.59.0 #1229

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

vthib
Copy link
Contributor

@vthib vthib commented Dec 1, 2024

The windows-sys crate has been bumped to 0.59.0 4 months ago. In my projcets, most of the crates using it has been updated but rustix, hence this PR to align it.

I followed the same changes as were done in #935 , that is bumping the errno version at the same time to make it use this new windows-sys version as well.

I didn't know whether to push this to main or 0.38, i figured main is probably the right one and a backport could be done, but tell me if you need me to target another branch.

Thanks!

@sunfishcode
Copy link
Member

Could you change this to allow either 0.52 or 0.59, as the errno crate does? That way it won't introduce duplicate window-sys dependencies for existing users that are using 0.52.

And yes, main is good. I'll backport it to 0.38 after it's landed.

The code is compatible with version 0.52.0 and version 0.59.0.

Additionally, bump errno to 0.3.10 to get this requirement on
errno as well.
@vthib vthib force-pushed the update-windows-sys-0.59 branch from 3db6f5c to 863f86a Compare December 3, 2024 22:00
@vthib
Copy link
Contributor Author

vthib commented Dec 3, 2024

Yes good idea, I updated the requirements

@sunfishcode sunfishcode merged commit 7a76ef5 into bytecodealliance:main Dec 4, 2024
45 checks passed
@sunfishcode
Copy link
Member

Thanks!

@vthib vthib deleted the update-windows-sys-0.59 branch December 6, 2024 13:55
sunfishcode pushed a commit that referenced this pull request Dec 8, 2024
The code is compatible with version 0.52.0 and version 0.59.0.

Additionally, bump errno to 0.3.10 to get this requirement on
errno as well.
sunfishcode pushed a commit that referenced this pull request Dec 8, 2024
The code is compatible with version 0.52.0 and version 0.59.0.

Additionally, bump errno to 0.3.10 to get this requirement on
errno as well.
@sunfishcode
Copy link
Member

This is now released in rustix 0.38.42.

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