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

lifenews.ru - add support to extract all videos #2482

Closed
spvkgn opened this issue Feb 28, 2014 · 3 comments
Closed

lifenews.ru - add support to extract all videos #2482

spvkgn opened this issue Feb 28, 2014 · 3 comments
Labels

Comments

@spvkgn
Copy link

@spvkgn spvkgn commented Feb 28, 2014

Some pages on lifenews.ru contains more than one video, for instance this page http://lifenews.ru/news/128115 with two videos and youtube-dl extract the first video only. Would be nice if you could add support to extract all videos.

$ youtube-dl -vg http://lifenews.ru/news/128115
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-vg', 'http://lifenews.ru/news/128115']
[debug] Encodings: locale 'UTF-8', fs 'UTF-8', out 'UTF-8', pref: 'UTF-8'
[debug] youtube-dl version 2014.02.26
[debug] Python version 2.7.3 - Linux-3.2.0-39-generic-i686-with-LinuxMint-13-maya
[debug] Proxy map: {}
http://lifenews.ru/static/posts/2014/2/128115/video.mp4

@jaimeMF jaimeMF added the request label Feb 28, 2014
dstftw added a commit that referenced this issue Feb 28, 2014
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Feb 28, 2014

Thanks for the report, fixed.
Hey, @phihag, is there some idiomatic way to handle those situations when there are several videos on the same page under the same URL, so that is not a playlist?

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Feb 28, 2014

Currently the only option apart from the playlist is to return a list (as you have done) with the info dicts. We could wrap it in a dictionary with "_type": "list"or something similar and an entries field, but I think the current solution is simpler. Do you have any idea for a better method?

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Feb 28, 2014

Current solution is simpler, but redundant. However, in general, each video may have it's own metadata and for this case that's ok. For generic solution I think, yes, wrapping different metadata fields into entries list and keeping common ones outside would be good.

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.