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

--add-metadata has some weird behavior #9650

Closed
sapphonie opened this issue May 31, 2016 · 2 comments
Closed

--add-metadata has some weird behavior #9650

sapphonie opened this issue May 31, 2016 · 2 comments
Labels

Comments

@sapphonie
Copy link

@sapphonie sapphonie commented May 31, 2016

Description of your issue, suggested solution and other information

M'kay, so apparently --add-metadata involves rewriting the ENTIRE FILE?

youtube-dl -iw --no-continue $URL -f bestvideo+bestaudio --add-metadata --merge-output-format mp4 -o "[%(upload_date)s] %(title)s" --prefer-ffmpeg --download-archive archive.txt --postprocessor-args "-strict -2 -threads 8 -c:v libx264 -crf 18 -preset veryslow -c:a aac -b:a 192k -movflags +faststart" gives me an output of example.mp4 , fully playable, without metadata. Then I see another copy, example.temp.mp4, that slowly grows to the size of example.mp4 and then replaces it. The output of youtube-dl during all of this is the standard [ffmpeg] Adding metadata to 'example.mp4'

Any way to fix this?

@TRox1972
Copy link
Contributor

@TRox1972 TRox1972 commented May 31, 2016

That isn't possible with ffmpeg; it always needs to construct a new file with the applied changes.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented May 31, 2016

This behavior is explicitly requested by you: --postprocessor-args is passed to any postprocessor involved including one that adds metadata. Remove any re-encoding from --postprocessor-args and it will be as fast as adding metadata.

@dstftw dstftw closed this May 31, 2016
@dstftw dstftw added the invalid label Jun 1, 2016
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.