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

NBC Nightly News #4884

Closed
ryandesign opened this issue Feb 6, 2015 · 3 comments
Closed

NBC Nightly News #4884

ryandesign opened this issue Feb 6, 2015 · 3 comments

Comments

@ryandesign
Copy link
Contributor

@ryandesign ryandesign commented Feb 6, 2015

Hello,

The supported sites page lists "NBCNews", however I am unable to download the NBC Nightly News, for example:

$ youtube-dl --version
2015.02.04
$ youtube-dl http://www.nbcnews.com/nightly-news/video/nightly-news-with-brian-williams-full-broadcast-february-4-394064451844
[generic] nightly-news-with-brian-williams-full-broadcast-february-4-394064451844: Requesting header
WARNING: Falling back on generic information extractor.
[generic] nightly-news-with-brian-williams-full-broadcast-february-4-394064451844: Downloading webpage
[generic] nightly-news-with-brian-williams-full-broadcast-february-4-394064451844: Extracting information
ERROR: Unsupported URL: http://www.nbcnews.com/nightly-news/video/nightly-news-with-brian-williams-full-broadcast-february-4-394064451844

NBC has decided to discontinue publishing their NBC Nighty News video podcast, so I am trying to use youtube-dl to build a new personal viewing solution for this content. Thanks for your help.

@rrooij
Copy link
Contributor

@rrooij rrooij commented Feb 10, 2015

I think that it's a problem with the _VALID_URL regex in NBCNewsIE:

_VALID_URL = r'''(?x)https?://www\.nbcnews\.com/
        ((video/.+?/(?P<id>\d+))|
        (feature/[^/]+/(?P<title>.+)))
        '''
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Feb 14, 2015

NBC nightly news will be supported in the next version, thanks for the report.

@dstftw dstftw closed this Feb 14, 2015
@phihag
Copy link
Contributor

@phihag phihag commented Feb 16, 2015

NBC support has been included in youtube-dl 2015.02.16. See our FAQ if you need help updating.

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