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

Put description metadata in the id3v2 "comment" field #4847

Closed
NelsonSR opened this issue Feb 2, 2015 · 2 comments
Closed

Put description metadata in the id3v2 "comment" field #4847

NelsonSR opened this issue Feb 2, 2015 · 2 comments
Labels

Comments

@NelsonSR
Copy link

@NelsonSR NelsonSR commented Feb 2, 2015

I've noticed the last days that now, using the --add-metadata option, youtube-dl retrieve the URL and the description of the videos ("comments" and "desc" fields, respectively), that's really nice, but it's completely pointless to put some metadata in a field that some video players won't display in a properties card.

For example, in VLC player (one of the most popular around) will display the "comments" field, but not the "desc" field, and for some kind of videos (screencasts, for example) that one is more important than the URL where the video is (or was) located. I don't know if iTunes display it.

So I suggest to put the description of the video in the comments field, and the URL in some other field like "purl" (podcast URL).

@jaimeMF jaimeMF added the request label Feb 2, 2015
@phihag phihag closed this in 2cf0eca Feb 2, 2015
@phihag
Copy link
Contributor

@phihag phihag commented Feb 2, 2015

Thanks, I've implemented exactly what you suggested, and it now shows up great in vlc. See our FAQ if you need help to update to youtube-dl 2015.02.03 or newer, where this is implemented.

@NelsonSR
Copy link
Author

@NelsonSR NelsonSR commented Feb 3, 2015

Yeah, I'm seeing it now and it looks great! thanks so much.

By the way, the "purl" field isn't being written (I don't need it, actually, but I thought i was part of the implementation).

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.