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

Add support for cbc.ca #6342

Closed
wants to merge 2 commits into from
Closed

Conversation

atomicdryad
Copy link
Contributor

No description provided.


webpage = self._download_webpage(url, name)
title = unescapeHTML(
self._search_regex('<title>\s*(.+?)\s*</title>', webpage, 'title'))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't _html_search_regexwork here?

@yhager
Copy link

yhager commented Aug 12, 2015

I'm interested in that too , and the tests seem to be working. Any chance this was a travis fluke and can be tried again?

@yan12125
Copy link
Collaborator

yan12125 commented Apr 9, 2016

Since #8497 landed, please move changes in youtube_dl/extractors/__init__.py to youtube_dl/extractors/extractors.py. Check step 5 of the new developer instructions for more information.

@yan12125 yan12125 mentioned this pull request Apr 9, 2016
yan12125 pushed a commit that referenced this pull request May 20, 2016
The URL http://www.cbc.ca/player/News/ID/2672225049/ (#6342) redirects
to http://www.cbc.ca/player/play/2672224672, while youtube-dl wasn't
able to handle it correctly.
@yan12125
Copy link
Collaborator

Since @remitamine has pushed his/her own implementation in 8b85ac3, this PR is no longer necessary.

@yan12125 yan12125 closed this May 20, 2016
TRox1972 referenced this pull request in TRox1972/youtube-dl May 20, 2016
The URL http://www.cbc.ca/player/News/ID/2672225049/ (#6342) redirects
to http://www.cbc.ca/player/play/2672224672, while youtube-dl wasn't
able to handle it correctly.
TRox1972 referenced this pull request in TRox1972/youtube-dl May 21, 2016
The URL http://www.cbc.ca/player/News/ID/2672225049/ (#6342) redirects
to http://www.cbc.ca/player/play/2672224672, while youtube-dl wasn't
able to handle it correctly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants