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

mp3 tagging doesn't work as one might expect #21785

Closed
Volker-Weissmann opened this issue Jul 14, 2019 · 2 comments
Closed

mp3 tagging doesn't work as one might expect #21785

Volker-Weissmann opened this issue Jul 14, 2019 · 2 comments
Labels

Comments

@Volker-Weissmann
Copy link

@Volker-Weissmann Volker-Weissmann commented Jul 14, 2019

Checklist

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

Description

If you run
youtube-dl --extract-audio --audio-format mp3 "https://youtu.be/Sqk3B2041uk" --output test.mp3
id3v2 -t "cooltitle" -a "coolartist" test.mp3
and open the resulting test.mp3 file using rythmbox, rythmbox doesn't know that the titel is "cooltitle" and that the artist is "coolartist".
But if you run
youtube-dl --extract-audio --audio-format mp3 "https://youtu.be/Sqk3B2041uk" --output "test.%(ext)s"
id3v2 -t "cooltitle" -a "coolartist" test.mp3
instead it works.
I think it is very unintuitive that the first version doesn't work. It would be nice if that would change, because I don't want other people to write the first one and then ask themselves for hours why this doesn't work.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jul 14, 2019

Output template is a template for downloaded file. Never hardcode ext for it.

@dstftw dstftw closed this Jul 14, 2019
@dstftw dstftw added the invalid label Jul 14, 2019
@Volker-Weissmann
Copy link
Author

@Volker-Weissmann Volker-Weissmann commented Jul 14, 2019

Output template is a template for downloaded file. Never hardcode ext for it.

It would be nice if this would be better explained in the manfile.

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.