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

json Property _filename - please add a property for the actually created filename #25914

Closed
schittli opened this issue Jul 6, 2020 · 0 comments
Closed
Labels

Comments

@schittli
Copy link

@schittli schittli commented Jul 6, 2020

Checklist

  • I'm reporting a broken site support issue
  • I've verified that I'm running youtube-dl version 2020.06.16.1
  • I've checked that all provided URLs are alive and playable in a browser
  • I've checked that all URLs and arguments with special characters are properly quoted or escaped
  • I've searched the bugtracker for similar bug reports including closed ones
  • I've read bugs section in FAQ

Verbose log

ERROR: youtube-dl.exe : WARNING: Requested formats are incompatible for merge and will be merged into mkv.

Description

It would be great if youtube-dl would return which file was actually created so that the created files could be post-processed.

Normally we can use --print-json and then read the property _filename.
But sometimes youtube-dl has to convert the downloaded file:

ERROR: youtube-dl.exe : WARNING: Requested formats are incompatible for merge and will be merged into mkv.

Then, the JSON property _filename does not reflect this change, and e.g. still displays the -mp4 extension:

_filename            : MyFile - 1920x1080.mp4

As a solution, a separate JSON property could be defined, for example created_filename

Thanks a lot for this great Tool!, it's great to create backups because even in Europe, censorship of harmless content is increasing.

Kind regards,
Thomas

@dstftw dstftw closed this Jul 6, 2020
@dstftw dstftw added the duplicate label Jul 6, 2020
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.