replace single reqwest::get with tcpstream, remove dependency

radareorg / r2pipe.rs

Rust crate for r2pipe

Other

45 stars 19 forks source link

replace single reqwest::get with tcpstream, remove dependency #42

Closed aemmitt-ns closed 2 years ago

aemmitt-ns commented 2 years ago

Checklist

[ ] Closing issues: #issue
[x] Mark this if you consider it ready to merge
[ ] I've added tests (optional)
[ ] I wrote some documentation

Description

Before I realized reqwest had been moved to a feature I wrote this to remove the need for the dependency. It's probably still better to have it this way though. This replaces the single call to reqwest::get with equivalent code using TcpStream.

aemmitt-ns commented 2 years ago

thank you! can we get a new version published? there are a lot of great changes not in the published version and it would be good to not need to pull in all the reqwest deps

meme commented 2 years ago

Published as 0.6.0