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

RFE: New Extractor: PBS #870

Closed
Baylink-gh opened this issue Jun 5, 2013 · 4 comments
Closed

RFE: New Extractor: PBS #870

Baylink-gh opened this issue Jun 5, 2013 · 4 comments

Comments

@Baylink-gh
Copy link
Author

@Baylink-gh Baylink-gh commented Dec 10, 2013

Reopening:

http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/

jra@princeton:/appl/media/tv> vget -tc --verbose http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-tc', '--verbose', 'http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/']
[debug] youtube-dl version 2013.12.10
[debug] Python version 2.7.2 - Linux-3.1.10-1.16-desktop-i686-with-SuSE-12.1-i586
[debug] Proxy map: {}
WARNING: Falling back on generic information extractor.
[generic] : Downloading webpage
[generic] : Extracting information
ERROR: Unsupported URL: http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type youtube-dl -U to update.
Traceback (most recent call last):
File "/usr/local/bin/vget/youtube_dl/YoutubeDL.py", line 471, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/vget/youtube_dl/extractor/common.py", line 134, in extract
return self._real_extract(url)
File "/usr/local/bin/vget/youtube_dl/extractor/generic.py", line 254, in _real_extract
raise ExtractorError(u'Unsupported URL: %s' % url)
ExtractorError: Unsupported URL: http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type youtube-dl -U to update.

@Baylink-gh
Copy link
Author

@Baylink-gh Baylink-gh commented Dec 10, 2013

FWIW, it does work ok on the base page: http://video.pbs.org/video/2365006249/

So maybe this doesn't need reopened. Thanks to whomever built it, BTW.

@phihag phihag reopened this Dec 10, 2013
@yasoob
Copy link
Contributor

@yasoob yasoob commented Feb 4, 2014

@phihag from what I saw both pages have same markup and hence same mechanism which means that we just have to add a new regex rule to catch these urls. What do you say?
The problem is that some of the urls are a lot of different from each other. For example : http://www.pbs.org/wgbh/masterpiece/watch-online/full-episodes/sherlock-s3-e3-his-last-vow/
http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/

@phihag phihag closed this in 22e7f1a Feb 4, 2014
@phihag
Copy link
Contributor

@phihag phihag commented Feb 4, 2014

Support for the article URLs like http://www.pbs.org/tpt/constitution-usa-peter-sagal/watch/a-more-perfect-union/ has been added in youtube-dl 2014.02.04. Type sudo youtube-dl -U to update.

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
3 participants
You can’t perform that action at this time.