Closed ryochin closed 5 years ago
hmm, I do not want to increase the dependencies.
You could probably make it optional. If Term::ProgressBar is installed already on the system to display progress bar?
No, this code is use Term::ProgressBar
by default.
Please have a look at the additional commit. Feel free to discard the issue if you would't like it.
@xaicron this no longer introduces any new dependencies. Did you have any further thoughts on it?
@ryochin do you have a moment to rebase this? I think we can merge and deploy it.
@ryochin do you have a moment to rebase this? I think we can merge and deploy it.
Here is pull request with rebase: https://github.com/xaicron/p5-www-youtube-download/pull/50
This has been rebased and now lives in d741addc3416658. Thanks @ryochin!
I guess it's slighty useful everyone can see ETA while long downloading period.