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

Metadata handling in README #14

Closed
ottre opened this issue Jul 25, 2018 · 4 comments
Closed

Metadata handling in README #14

ottre opened this issue Jul 25, 2018 · 4 comments

Comments

@ottre
Copy link

ottre commented Jul 25, 2018

I think you should note in the README that metadata handling isn't fully implemented. In particular, there's no way to get the URL of the currently playing file.

Please correct me if I'm wrong, once the TrackList interface is added you will be able to get the URL by calling TrackList.GetTracksMetadata.

This is useful info for people who want to communicate with multiple instances of mpv playing videos.

@ottre ottre changed the title Metadata handling Metadata handling in README Jul 25, 2018
@hoyon
Copy link
Owner

hoyon commented Jul 26, 2018

The MPRIS spec only requires the mpris:id metadata property to be implemented, and any others are up to the implementer to choose.

The URL property was not implemented because there is no easy way to get the URI of the current file with the mpv API, so must be constructed manually. I might give this a go though.

Regarding the TrackList interface, I don't currently intend on implementing this as it would require a lot of extra work, and I'm not sure if anything actually uses it.

@hoyon hoyon closed this as completed in 5413e5d Aug 18, 2018
@hoyon
Copy link
Owner

hoyon commented Aug 18, 2018

This is my initial attempt at this. It might not work in all cases, so let me know if there are any paths it doesn't work for.

@timakro
Copy link

timakro commented Apr 14, 2019

This works nicely with gpodder. Could we get a new release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants
@hoyon @timakro @ottre and others