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 with windows-sys #534

Closed
russellbanks opened this issue Jan 1, 2024 · 5 comments · Fixed by #536
Closed

Replace winapi with windows-sys #534

russellbanks opened this issue Jan 1, 2024 · 5 comments · Fixed by #536

Comments

@russellbanks
Copy link
Contributor

winapi hasn't been updated in over 3 years. windows-sys is an official Microsoft crate that is actively maintained. I can have a go at the migration and create a pull request if this is an acceptable change.

@zeenix
Copy link
Contributor

zeenix commented Jan 1, 2024

Sounds good! However, perhaps best use the high-level windows crate?

@russellbanks
Copy link
Contributor Author

Unless there's anything specifically needed in the higher level crate, windows-sys is generally preferable in libraries because it has faster compile times and doesn't depend on std. Anecdotally, it's also more similar to winapi's API and has been very simple to migrate - hwchen/keyring-rs#159.

windows-or-windows-sys

@zeenix
Copy link
Contributor

zeenix commented Jan 1, 2024

@russellbanks thanks for explaining. Sure, let's go for windows-sys.

zeenix added a commit that referenced this issue Jan 2, 2024
⬆️ Migrate to windows-sys from winapi (#534)
@zeenix
Copy link
Contributor

zeenix commented Feb 11, 2024

@russellbanks thanks for explaining. Sure, let's go for windows-sys.

@russellbanks Hi, I'm getting a strange panic from windows-sys against nightly (only) in busd tests. Looking at the relevant code in zbus, I don't see what's causing it. Also, it makes me rethink using -sys crate directly.

Ideas?

@zeenix
Copy link
Contributor

zeenix commented Feb 11, 2024

@russellbanks Hi, I'm getting a strange panic from windows-sys against nightly (only) in busd tests. Looking at the relevant code in zbus, I don't see what's causing it. Also, it makes me rethink using -sys crate directly.

Ideas?

Filed an issue and I think I found a fix.

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 a pull request may close this issue.

2 participants