rust-libp2p icon indicating copy to clipboard operation
rust-libp2p copied to clipboard

feat: remove async_std support in TCP crate

Open gitToki opened this issue 9 months ago • 8 comments

ref https://github.com/libp2p/rust-libp2p/issues/5935

crate to update: swarm, mDNS, and the transports TCP, QUIC and DNS.

gitToki avatar Mar 25 '25 13:03 gitToki

Can we not soft deprecate and just hard deprecate? CC @elenaf9 @dariusc93

@gitToki initially did that, but I asked for a soft deprecation. I think a soft deprecation makes sense here because we might still have users that use async_std. It would a) give them time to object before we do a hard removal and b) offer a transition period for them.

Any specific reason why you'd prefer a direct removal @jxs?

elenaf9 avatar Mar 26 '25 00:03 elenaf9

Can we not soft deprecate and just hard deprecate? CC @elenaf9 @dariusc93

Would there be any reason to suddenly do a hard deprecation vs soft? Imo, I think we should do a soft (though I am not against a hard one) to give users a time to switch or figure out plans on migration (or supporting the executor themselves for the respected protocols and utils).

dariusc93 avatar Mar 26 '25 00:03 dariusc93

Hi!

Any specific reason why you'd prefer a direct removal @jxs?

the reason is more work, double the PR's for something that has been declared as unmaintained, I feel redundant to have a version for users to remove their usage when they probably are already removing, added to the fact that I have seen soft deprecations lost in time. But if you two agree that we should do soft I cede :)

jxs avatar Apr 03 '25 15:04 jxs

updated, should be good. @dariusc93 please let me know if everything look good to you

gitToki avatar Apr 13 '25 19:04 gitToki

This pull request has merge conflicts. Could you please resolve them @gitToki? 🙏

mergify[bot] avatar Jun 04 '25 13:06 mergify[bot]

Is it good to you now @elenaf9 ? 🫡

gitToki avatar Jun 05 '25 16:06 gitToki

This pull request has merge conflicts. Could you please resolve them @gitToki? 🙏

mergify[bot] avatar Jun 09 '25 09:06 mergify[bot]

dcutr and mplex fail but only the test have been updated so no version bump. It should be good now

gitToki avatar Jun 10 '25 19:06 gitToki

This pull request has merge conflicts. Could you please resolve them @gitToki? 🙏

mergify[bot] avatar Jun 21 '25 19:06 mergify[bot]