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

Upgrade ioctl-sys to 0.8.0 #100

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Upgrade ioctl-sys to 0.8.0 #100

merged 1 commit into from
Jun 3, 2024

Conversation

faern
Copy link
Member

@faern faern commented May 29, 2024

ioctl-sys has no dependencies, so this does not affect the dependency tree more than this direct dependency. There were no direct needs to upgrade except that it's nice to just go with the latest from time to time.

ioctl-sys added support for OpenBSD. We have not tried this crate on OpenBSD, but since that OS has PF, this might make this crate work on OpenBSD.

I did read the actual diff between 0.6.0 and 0.8.0. Not much noteworthy except the support for OpenBSD.


This change is Reviewable

@faern faern requested a review from Serock3 May 29, 2024 20:43
Copy link

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

ioctl-sys has no dependencies, so this does not affect the dependency
tree more than this direct dependency. There were no direct needs to
upgrade except that it's nice to just go with the latest from time to
time.

ioctl-sys added support for OpenBSD. We have not tried this crate on
OpenBSD, but since that OS has PF, this *might* make this crate work on
OpenBSD.
@faern faern merged commit ce13c03 into main Jun 3, 2024
7 of 8 checks passed
@faern faern deleted the upgrade-ioctl-sys branch June 3, 2024 22:25
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