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

Feature gate tokio features on mdns crate. #1831

Merged
merged 1 commit into from Nov 15, 2022
Merged

Conversation

jxs
Copy link
Contributor

@jxs jxs commented Nov 15, 2022

addresses #1830

@djc
Copy link
Collaborator

djc commented Nov 15, 2022

Thanks!

@jxs
Copy link
Contributor Author

jxs commented Nov 15, 2022

Does the failing test relate with this PR?
Don't know if it helps but running cargo make all-features on my Linux x395 5.15.72-1-lts #1 SMP machine with OpenSSL 1.1.1q 5 Jul 2022 all tests pass, including test_tls_client_stream_ipv4

@djc
Copy link
Collaborator

djc commented Nov 15, 2022

Yeah, the failure seems unrelated. @bluejekyll any ideas?

@jxs
Copy link
Contributor Author

jxs commented Nov 15, 2022

seems to have passed now :)

@djc djc merged commit fe4a95f into hickory-dns:main Nov 15, 2022
@jxs jxs deleted the fix-mdns branch November 15, 2022 19:38
@thomaseizinger
Copy link

Is there any chance we can get a patch-release with this in it? :)

@djc
Copy link
Collaborator

djc commented Nov 20, 2022

I can't do releases, only @bluejekyll can and he seems quite busy recently. Would need to start a 0.22.x maintenance branch because AFAICT main contains some incompatible changes.

mergify bot pushed a commit to libp2p/rust-libp2p that referenced this pull request Sep 4, 2023
On #3102 we left the `tokio-runtime` feature because of hickory-dns/hickory-dns#1830, meanwhile hickory-dns/hickory-dns#1831 was merged and `0.23.0` was released with it, so we no longer require the `tokio-runtime` feature.

Pull-Request: #4418.
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.

None yet

3 participants