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

Enhancement request: add option to ignore WebM (or add "bestmp4", "bestwebm", etc. format names) #5015

Closed
zmwangx opened this issue Feb 20, 2015 · 2 comments

Comments

@zmwangx
Copy link

@zmwangx zmwangx commented Feb 20, 2015

Most of the time bestvideo on YouTube is H.264 video in MPEG-4 container, but occasionally bestvideo is VP9 video in WebM container, and this has been screwing me on every encounter. Problems:

  • WebM container doesn't support AAC (only Opus and Ogg Vorbis are supported), so when bestvideo is WebM, bestvideo+bestaudio always fails since youtube-dl asks FFmpeg to mux VP9 and AAC into a WebM container, and it simply can't.
  • I'm not interested in VP9 in general, since it's not supported on iOS. I can afford larger files with H.264.

Therefore, I would like to be able to ignore WebM altogether. I think a good idea would be to add special format codes bestmp4, bestwebm, etc. so that the user can target the specific format they want, without resorting to doing youtube-dl -F first and filtering the results manually.

@phihag
Copy link
Contributor

@phihag phihag commented Feb 20, 2015

You can simply use -f "bestvideo[ext=mp4]" already. I do not see the need to add any more codes.

Also bear in mind that bestvideo is not the same as best. It will pick the best video-only format.

@phihag phihag closed this Feb 20, 2015
@zmwangx
Copy link
Author

@zmwangx zmwangx commented Feb 21, 2015

Uh apparently filtering by properties was implemented only a month ago in 083c9df and I didn't read the updated docs. Sorry.

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