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

Replace winapi crate with windows-sys #14

Closed
kgrech opened this issue Dec 9, 2022 · 5 comments
Closed

Replace winapi crate with windows-sys #14

kgrech opened this issue Dec 9, 2022 · 5 comments
Assignees

Comments

@kgrech
Copy link

kgrech commented Dec 9, 2022

Depending on winapi crate is not the best idea:

  • It is not maintained
  • The source code of it contains large number of precompiled binaries (see here). It is not clear how were they build, what is the license of them and what binary code they contain
  • There is an official alternative windows crate supported by Microsoft.
  • Other projects are migrating already (e.g. socket2, remove-dir-all, mio and also parking_lot, wasmtime, and others)

Would you consider the migration?

@fdncred
Copy link

fdncred commented Dec 9, 2022

Would you consider the migration?

Yes, probably so. Although my initial concern would be increased compile time or increased dependencies. I'm familiar with Microsoft's crate and agree it's probably the direction we should go. I'm just hoping it's a change for the better.

@Sytten
Copy link

Sytten commented Feb 24, 2023

At this point most people are already using the official windows-sys crate anyway since a lot of projects moved to it, so I would say not to worry too much about compile time or increased dependencies.

@fdncred
Copy link

fdncred commented Feb 24, 2023

LOL. Just don't worry about it. :)

One of the problems is that the windows crate is moving very fast. So, that means lots of versions. I also agree that people are migrating to it like they should. The problem comes in where we have 10 versions of the windows crates because people don't keep their projects up to date. That's especially a problem in crates like this one that don't get updated very frequently. i.e. It'll get behind fast. But that's the age-old story with crates.

Having said all that, like I said before, we'd probably accept a PR to move to the windows crate.

@kgrech
Copy link
Author

kgrech commented Feb 25, 2023

Having said all that, like I said before, we'd probably accept a PR to move to the windows crate.

I am happy to contribute it then

@sholderbach
Copy link
Member

Thanks for picking that up and moving the ecossystem forward @kgrech!

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

No branches or pull requests

4 participants