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 to 0.59.0 #369

Closed
wants to merge 3 commits into from
Closed

Conversation

chantra
Copy link

@chantra chantra commented Oct 2, 2024

Simple windows-sys version bump to the latest published crate version.

@Wojtek242
Copy link
Collaborator

Hi @chantra, thanks for the PR, but based on the build outputs: https://github.com/rust-pcap/pcap/actions/runs/11135749919/job/32117440201?pr=369 you will need to ammend the code as well. Let me know if you need help.

You can ignore the lints as that is currently also a problem on main.

@chantra
Copy link
Author

chantra commented Oct 27, 2024

Thanks @Wojtek242 ,

I will update the PR accordingly. I think that now that you authorized this PR to run the GitHub actions, update should do too, so this may help. I don’t have a windows setup handy, but I am fairly confident I should be able to cross-compile easily with cargo.

@chantra
Copy link
Author

chantra commented Oct 28, 2024

@Wojtek242 ok, it seems I can't get the CI to run without approval.

This now builds with:

cargo build --target=x86_64-pc-windows-gnu

Since windows-sys-0.45.0,
`windows_sys::Win32::Networking::WinSock::ADDRESS_FAMILY` are u16 instead
of u32.
windows_sys::Win32::Foundation::HANDLE type
has changed since microsoft/windows-rs@d331301

update the tests accordingly.
@qrnch-jan
Copy link
Contributor

I apologize for PR-sniping you; I didn't see this PR before making #378

Your updated is incomplete -- it's missing fixes that are unveiled by the capture-stream feature (always build/test using --all-features just to be sure). You should merge those fixes from my PR to yours and then I'll close mine.

@chantra
Copy link
Author

chantra commented Dec 18, 2024

I apologize for PR-sniping you; I didn't see this PR before making #378

Your updated is incomplete -- it's missing fixes that are unveiled by the capture-stream feature (always build/test using --all-features just to be sure). You should merge those fixes from my PR to yours and then I'll close mine.

Thanks @qrnch-jan for the heads up. I am totally fine with your PR making it through.
I will close this one in favor of your.

@chantra chantra closed this Dec 18, 2024
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.

3 participants