Re: [PATCH] upnp: add support for thumbnails

[Jens Georg <mail jensge org>]

On Thu, 2011-01-20 at 12:59 +0100, Guillaume Emont wrote:
> On 20/01/2011 10:35, Jussi Kukkonen wrote:
> > My tracker/rygel installation seems to be acting up after all the
> > testing at the moment... but rygel should give you a albumArtURI in the
> > right conditions, I believe. The DIDL excerpt below is from rygel, I
> > think (not 100% sure about that though).
> Managed to get that with ryhtmbox's upnp plugin, once you've played a
> song that has cover art (does not seem to be persistent over sessions
> though). Also, it turns out that if (and only if) it's available with
> rhythmbox, it's available with rygel as well, from my experience. Also
> it's weird that in both cases, thumbnails are only available for
> individual tracks, and only the one that have been played, and not for
> the album nor for the non-played tracks in the album...

Let me clarify this a bit :)

As for the availability of the icon, as I said in a previous post, rygel
relies on the MediaArtStorageSpec
(http://live.gnome.org/MediaArtStorageSpec), and rhythmbox is so kind to
store its album art also according in that spec (other software doing
this are banshee or tracker).

Rygel doesn't do cover download, but with the MediaExport back-end it is
able to extract embedded album art and put in that folder as well, so
for those files it _should_ be available without rhythmbox helping (if
not, file a bug, please). For the tracker back-end tracker does that
job.

As for the availability of the picture for single tracks only and not
the whole album, that depends on how rhythmbox saves the image; judging
from your description it seems that rhythmbox saves this as a track
album-art and according to the spec we then have no chance of deducing
the album from it. This is easy to check, the file name should start
with "track-". If it doesn't start with track but with "album-" then
please file a bug against rygel.

> Oh, and of course, the url provided by rhythmbox doesn't work, and the
> one provided by rygel is a file://.

It is offered to you as "file://" since you connect from localhost. If
you connect on any other interface, you should get a http proxy url.

> Anyway, I managed to implement that and test it a bit, it's a one-liner
> with gupnp_didl_lite_object_get_album_art().
> > I'll check if Rygel gives you large images (as I expect it to) later as
> > well. I don't think I've actually seen pngs used but if it's in the DLNA
> > profiles we should try to support it: It would sound smart to use png as
> > thumbnail format for png images, and I expect that does happen in reality.
> In my experience it didn't, but my ~/.thumbnails/large is empty, will
> try more with that, and try to support PNG_ and _SN, which will make me
> refactor the stuff once again ;)

Album art is stored in ~/.cache/media-art.