-
Notifications
You must be signed in to change notification settings - Fork 111
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
youtube-dl in docker is outdated #34
Comments
@ko-lem do you have that forked container published anywhere? I'm looking for a good youtube-dl webui container to use myself. |
@yllekz not yet. I'll look into doing so this weekend |
Greetings! Any update on adding Dockerfile to this repo? |
I'll check this weekend |
I will say sorry for any bothering problems that caused by lacking of maintenance. I have less spare time to devote to personal projects after I made a shift to a new company 3 years before. This project is open to anyone, that any pull request is welcome, please feel free to commit. I migrate dockerfile from another repo to here: https://github.com/d0u9/youtube-dl-webui/tree/master/dockerfile Sorry again. |
No worries. I've also been too busy to make the fix I wanted. |
Posted some updates on Dockerfile: #40 |
wouldn't it be more reliable to add a scheduled routine, maybe a cron job that updates youtube-dl daily? |
I made a modification into entrypoint to upgrade youtube-dl but to persist it, between container restart, we need to add a volume to /usr/local/bin/ just add -e UPGRADE=true to update at startup. #41 |
There's no need to persist it. Just add a youtube-dl update to the startup process and cron job it to update every day. that way whether you start the container or leave it run for a month, its always up to date. and you don't need to cut new images every other day. |
Nice project :)
I used your published docker container but downloads always failed. I debugged and found that the
youtube-dl
inside is outdated (year 2018).It would be nice if you could rebuild the container and publish new version. For now, I'm just making my own container to run your project.
The text was updated successfully, but these errors were encountered: