-
Notifications
You must be signed in to change notification settings - Fork 44
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
System call to Python dependency #235
Comments
Looks promising, we'll look into it. Thank you! |
This seems interesting. I want to check if rustube can replace yt-dlp. Can I try to implement it or are you in the middle of trying it? |
@mustafasegf go ahead, give it a try and if it works tag us in a PR |
little but update on the issue. Rustube right now can't send stream data to stdout. The metadata it get also not complete yet. So i can't integrate rustube for now. I'm thinking to try add those feature to rustube first |
While we do not have a rust-native alternative: We don't really need Python. We can shift to using the binaries directly. Should be as easy as adding them to the PATH and we're good to go. I suppose that the Python install is easier for anyone to replicate, but we can just as easily create a curl command to fetch |
🧐 Rationale
Remove dependency on Python's
yt-dlp
.📝 Description
Currently,
yt-dlp
package is being used to download YouTube videos, both requiring a Python dependency and a system call.parrot/src/sources/youtube.rs
Line 60 in 437e8b2
However, there is a crate for Rust that achieves the same purpose. It's still a work in progress, but might be a fitting replacement in the future.
The text was updated successfully, but these errors were encountered: