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

[discussion] sites covered by generic extractor #21317

Closed
ealgase opened this issue Jun 6, 2019 · 2 comments
Closed

[discussion] sites covered by generic extractor #21317

ealgase opened this issue Jun 6, 2019 · 2 comments
Labels

Comments

@ealgase
Copy link
Contributor

@ealgase ealgase commented Jun 6, 2019

The problem

Many sites work with youtube-dl's generic extractor. However, the generic extractor often misses metadata, and doesn't often work for playlists. This has resulted in some frustrating situations, such as #21142 (although that is definitely not the only one).

The question

Should youtube-dl support sites that the generic extractor works for, to add support for metadata and playlists?

@ealgase ealgase added the question label Jun 6, 2019
@Skye000
Copy link

@Skye000 Skye000 commented Jun 6, 2019

Yes.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jun 7, 2019

The general rule is not to add separate extractors (until pressing need) if the generic one is capable of handling.

@dstftw dstftw closed this Jun 7, 2019
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.