search

learningequality / ka-lite-content-packs

BSD 2-Clause "Simplified" License
1 stars 6 forks source link

Do not stop on download errors if the video does not exist. #72

Closed mrpau-eduard closed 7 years ago

mrpau-eduard commented 7 years ago

Summary

If the video does not exist, ignore error and continue downloading the subtitles. Sample output below:

INFO:root:trying to download subtitle for UjU0tpulIQ8
[youtube] UjU0tpulIQ8: Downloading webpage
[youtube] UjU0tpulIQ8: Downloading video info webpage
ERROR: UjU0tpulIQ8: YouTube said: This video does not exist.
INFO:root:No available subtitle for (UjU0tpulIQ8)
INFO:root:trying to download subtitle for Xcrco59p40o
[youtube] Xcrco59p40o: Downloading webpage
[youtube] Xcrco59p40o: Downloading video info webpage
[youtube] Xcrco59p40o: Extracting video information
[youtube] Xcrco59p40o: Downloading MPD manifest
[info] Writing video subtitles to: /Users/mrpau-eduard/content-pack-maker/contentpacks/build/subtitles/en/Xcrco59p40o.en.vtt
INFO:root:Rename /Users/mrpau-eduard/content-pack-maker/contentpacks/build/subtitles/en/Xcrco59p40o.en.vtt -> /Users/mrpau-eduard/content-pack-maker/contentpacks/build/subtitles/en/Xcrco59p40o.vtt

Issues addressed

66

mrpau-eduard commented 7 years ago

It's a small fix I will merge this one and rebuild the language pack.