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

Consider moving from winapi to windows-sys #537

Open
workingjubilee opened this issue Jun 29, 2023 · 5 comments
Open

Consider moving from winapi to windows-sys #537

workingjubilee opened this issue Jun 29, 2023 · 5 comments

Comments

@workingjubilee
Copy link
Contributor

workingjubilee commented Jun 29, 2023

This is like #535 except it is asking a slightly different question but with the same goal: would migrating from winapi offer a more maintainable crate? If so, we should probably do it.

@workingjubilee workingjubilee changed the title Consider moving from winapi to windows Consider moving from winapi to windows-sys Jun 29, 2023
@atouchet
Copy link
Contributor

atouchet commented Jul 1, 2023

I believe @kennykerr was doing some work on this previously: #494

@kennykerr
Copy link

I think it's about time if everyone is in agreement - there were some questions in #494 but the std library has since moved to windows-rs as @alexcrichton suggested thanks to @ChrisDenton and the libs have been working reliably so I would image the concerns that @thomcc had would be satisfied.

@workingjubilee
Copy link
Contributor Author

I'd recommend getting that solved first, effectively deleting library/std/src/sys/windows/c.rs, and then once that's migrated this should be able to trivially follow.

Yeah I remember reviewing some of the PRs that were part of that cleanup.

I'm game to see that PR rebased and reopened, if you would like.

@ChrisDenton
Copy link
Contributor

I just started looking at this. We currently use our own definitions and have a whole nest of macros to confirm they match the ones in the winapi crate (with exceptions for known issues). windows-bindgen (soon riddle) allows generating bindings so we can greatly simplify things here.

@michaelwoerister
Copy link
Member

I'm currently looking into modifying the debuginfo search path so that PDBs next to the binary can be found. Being able to use windows-sys would greatly simplify that task, I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants