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

Use pnet datalink directly #24

Merged
merged 5 commits into from
Oct 8, 2024
Merged

Conversation

negezor
Copy link
Contributor

@negezor negezor commented Mar 31, 2024

Looking at my Cargo.lock I found a significant number of packages that pnet pulls, although only one standalone module is needed. This will also get rid of the syn v1 dependency. And I also simplified the code of the private_ipv4 function 😅

@negezor
Copy link
Contributor Author

negezor commented Apr 14, 2024

@bahlo gentle ping

@HigherOrderLogic
Copy link

Hola @bahlo! Any chance to got this merge soon?

Copy link
Owner

@bahlo bahlo left a comment

Choose a reason for hiding this comment

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

nice, thank you for the contribution!

@bahlo bahlo merged commit f5bd510 into bahlo:main Oct 8, 2024
4 checks passed
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