Bug 36961 - Playlist information is missing for Android devices
Summary: Playlist information is missing for Android devices
Status: RESOLVED FIXED
Alias: None
Product: media-player-info
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Martin Pitt
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-08 04:42 UTC by Bernd Schlapsi
Modified: 2011-05-17 00:21 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Bernd Schlapsi 2011-05-08 04:42:50 UTC
I think the playlist information is missing for (all) the Android devices. I just have two of them and so I only send the diff for this two mdi files

udevadm info --query=all --path=/devices/pci0000:00/0000:00:1a.0/usb1/1-1/1-1.6
P: /devices/pci0000:00/0000:00:1a.0/usb1/1-1/1-1.6
N: bus/usb/001/017
S: char/189:16
E: UDEV_LOG=3
E: DEVPATH=/devices/pci0000:00/0000:00:1a.0/usb1/1-1/1-1.6
E: MAJOR=189
E: MINOR=16
E: DEVNAME=/dev/bus/usb/001/017
E: DEVTYPE=usb_device
E: DRIVER=usb
E: PRODUCT=18d1/4e22/227
E: TYPE=0/0/0
E: BUSNUM=001
E: DEVNUM=017
E: SUBSYSTEM=usb
E: ID_VENDOR=Samsung
E: ID_VENDOR_ENC=Samsung
E: ID_VENDOR_ID=18d1
E: ID_MODEL=Nexus_S
E: ID_MODEL_ENC=Nexus\x20S
E: ID_MODEL_ID=4e22
E: ID_REVISION=0227
E: ID_SERIAL=Samsung_Nexus_S_3935AC4B5B4100EC
E: ID_SERIAL_SHORT=3935AC4B5B4100EC
E: ID_BUS=usb
E: ID_USB_INTERFACES=:080650:ff4201:
E: ID_MEDIA_PLAYER=google_nexus-s
E: DEVLINKS=/dev/char/189:16


udevadm info --query=all --path=/devices/pci0000:00/0000:00:1a.0/usb1/1-1/1-1.5
P: /devices/pci0000:00/0000:00:1a.0/usb1/1-1/1-1.5
N: bus/usb/001/018
S: char/189:17
E: UDEV_LOG=3
E: DEVPATH=/devices/pci0000:00/0000:00:1a.0/usb1/1-1/1-1.5
E: MAJOR=189
E: MINOR=17
E: DEVNAME=/dev/bus/usb/001/018
E: DEVTYPE=usb_device
E: DRIVER=usb
E: PRODUCT=bb4/c99/100
E: TYPE=0/0/0
E: BUSNUM=001
E: DEVNUM=018
E: SUBSYSTEM=usb
E: ID_VENDOR=HTC
E: ID_VENDOR_ENC=HTC
E: ID_VENDOR_ID=0bb4
E: ID_MODEL=Android_Phone
E: ID_MODEL_ENC=Android\x20Phone
E: ID_MODEL_ID=0c99
E: ID_REVISION=0100
E: ID_SERIAL=HTC_Android_Phone_HT97ML906443
E: ID_SERIAL_SHORT=HT97ML906443
E: ID_BUS=usb
E: ID_USB_INTERFACES=:080650:ff4201:
E: ID_MEDIA_PLAYER=htc_desire_hero
E: DEVLINKS=/dev/char/189:17
Comment 1 Aaron Whitehouse 2011-05-13 18:34:47 UTC
Hi,

I have an HTC Desire (lsusb: 0bb4:0ff9), so presumably:
http://cgit.freedesktop.org/media-player-info/tree/media-players/htc_desire_hero_evo-4g_legend.mpi

Rhythmbox saves playlists in the root folder. It still works, but it isn't as nice.

My (uneducated) guess is that we need a:
[storage]
PlaylistPath=PLAYLIST/%File;Playlist/%File
Comment 2 Bernd Schlapsi 2011-05-14 02:06:34 UTC
On my two Android devices there is still a Playlist folder I don't created by myself. This folder is in SDCARD/Music/Playlists. So I would say the setting should look like:

[storage]
PlaylistPath=Music/Playlists;PLAYLIST/%File;Playlist/%File

I think it's important that these settings exists in the mdi-files and it's not that import which folder is defined in the settings.
So far as I can tell the build in Music player (and other player app I tested) on Android searches for playlist files on the whole sdcard
Comment 3 Martin Pitt 2011-05-14 06:16:34 UTC
Thank you! I added that to all Android devices now:

http://cgit.freedesktop.org/media-player-info/commit/?id=918643726ef70d68f12193708ad986b11b59189f
Comment 4 Bernd Schlapsi 2011-05-15 05:41:42 UTC
Thank you Martin for the commit.

But I have one further question. What's the difference between
>> PlaylistPath=Playlists/%File
and
>> PlaylistPath=Playlist/;Playlists/
The first one I found in iriver_e30.mpi, the second one in archos_5imt.mpi

What does the "%File" is standing for? Because in my last comment I mixed up the definition of the PlaylistPath setting with both versions. 

I couldn't found anything about the %FILE in the README file of the media-player-info folder:
http://cgit.freedesktop.org/media-player-info/tree/README
Comment 5 Martin Pitt 2011-05-17 00:21:25 UTC
I checked this with Christophe Fergeau, and both Rhythmbox and Banshee now completely ignore %File. Seems originally PlaylistPath pointed to files (it can still do so), but modern players support multiple playlists, so it now can also point to directories. Originally "Dir/%File" meant "any playlist file in Dir/", but "Dir/" is equivalent to that.

I now removed all %File instances from git head.

Thanks for pointing out!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.