Skip to content
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

when --write-srt fails, youtube-dl exits #639

Closed
tianon opened this issue Jan 17, 2013 · 3 comments
Closed

when --write-srt fails, youtube-dl exits #639

tianon opened this issue Jan 17, 2013 · 3 comments

Comments

@tianon
Copy link

@tianon tianon commented Jan 17, 2013

On line 464 of InfoExtractors.py, writesubtitles will cause the video downloading to be skipped if downloading the subtitles fails. Shouldn't that be more of a warning than a fatal error? That way, --write-srt could be specified all the time, and only apply if subtitles are actually available and properly downloadable.

@tianon
Copy link
Author

@tianon tianon commented Jan 17, 2013

As an example, downloading "https://www.youtube.com/watch?v=cgoVvqZ6dfA" with --write-srt fails with:
WARNING: unable to download video subtitles: HTTP Error 400: Bad Request (not sure if it actually has valid, downloadable subtitles, but it does fail and prevent downloading of the video).

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Jan 18, 2013

We are in progress of completely reviewing our concepts of WARNINGS and ERRORS, but meanwhile the behavior is correct: missing subtitles trigger a WARNING and that stops the execution only if you have not specified -i (--ignore-errors) on the command line.

I think that almost everybody uses (or wants) -i so eventually we will make it default.

Thanks for reporting! Happy downloading.

@tianon
Copy link
Author

@tianon tianon commented Jan 18, 2013

Ah, that makes perfect sense. Didn't notice -i, or probably wouldn't have even made this ticket. Thanks for clarifying. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.