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

Remove output metadata options that do not apply #24144

Closed
HASJ opened this issue Feb 26, 2020 · 2 comments
Closed

Remove output metadata options that do not apply #24144

HASJ opened this issue Feb 26, 2020 · 2 comments
Labels

Comments

@HASJ
Copy link

@HASJ HASJ commented Feb 26, 2020

Checklist

  • I'm reporting a feature request
  • I've verified that I'm running youtube-dl version 2020.02.16
  • I've searched the bugtracker for similar feature requests including closed ones

Description

This is the output argument I am using:
-o "Downloads\Videos\%(extractor)s - %(uploader)s-%(upload_date)s-%(playlist)s #%(playlist_index)s - %(title)s [%(id)s].%(ext)s"

I would like for youtube-dl to, instead of always using this and then later requiring me to remove all the NA-NA-NA from files without the tags, know if the downloaded video actually has the tags and do a if check.

For example: https://v.redd.it/ps7wzmoq02j41 outputs Reddit - Artem10397-20200225-NA #NA - Russian cyberpunk. [ps7wzmoq02j41].mp4.
My feature proposal would make youtube-dl automatically rename it to Reddit - Artem10397-20200225- Russian cyberpunk. [ps7wzmoq02j41].mp4

@HASJ HASJ added the request label Feb 26, 2020
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Feb 27, 2020

That's technically impossible to implement what you request in your example. The max that can be is inserting empty strings instead of NA.

@dstftw dstftw closed this Feb 27, 2020
@canlang
Copy link

@canlang canlang commented Apr 20, 2020

That's technically impossible to implement what you request in your example. The max that can be is inserting empty strings instead of NA.

Are you saying that it's now possible to change to an empty string?
I agree with the @HASJ 's intention. It would be nice to change the NA statement with conditional or OR operation.

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.