After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 583934 - Rhythmbox ignores portable_audio_player interface
Rhythmbox ignores portable_audio_player interface
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Removable Media
0.12.x
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-05-26 19:51 UTC by Patryk Zawadzki
Modified: 2009-05-31 13:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Debugging output (31.62 KB, text/plain)
2009-05-31 10:18 UTC, Patryk Zawadzki
Details

Description Patryk Zawadzki 2009-05-26 19:51:34 UTC
My phone (HTC Dream) reports:

$ hal-device | grep audio
  portable_audio_player.access_method.protocols = { 'storage' } (string list)
  portable_audio_player.output_formats = { 'audio/x-ms-wma', 'audio/3gpp', 'audio/mp4', 'audio/aac', 'audio/vnd.rn-realaudio', 'audio/x-midi', 'audio/x-wav', 'audio/mpeg' } (string list)
  portable_audio_player.audio_folders = { 'Music/' } (string list)
  portable_audio_player.storage_device = '/org/freedesktop/...'  (string)
  info.capabilities = { 'storage', 'block', 'portable_audio_player', 'access_control' } (string list)
  info.category = 'portable_audio_player'  (string)

Rhythmbox happily ignores this data, including portable_audio_player.audio_folders. This results in rhythmbox grinding to a halt when it tries to index ~3GB of git trees in Code/ directory of my phone.

Rhythmbox should respect the audio_folders (and possibly output_formats) key when present.
Comment 1 Jonathan Matthew 2009-05-31 05:05:32 UTC
It works perfectly for me.  Output from 'rhythmbox -D generic' might help.
Comment 2 Patryk Zawadzki 2009-05-31 10:18:56 UTC
Created attachment 135651 [details]
Debugging output
Comment 3 Patryk Zawadzki 2009-05-31 10:20:42 UTC
See the attachment, it seems the directory is only obeyed for songs, not playlists. Add some gigs of source code with bazillions of small files and it soon gets ugly :)
Comment 4 Jonathan Matthew 2009-05-31 11:02:23 UTC
Indeed.. I don't think we really thought about what we should do when playlist support information is missing.  In this case, we don't have a playlist path spec or playlist formats, so it probably doesn't make sense to go looking for playlists at all.
Comment 5 Jonathan Matthew 2009-05-31 13:07:34 UTC
fix committed as 0c9d36ecc5d8866e69cdbadbe29495fc243e9817

This didn't make it into 0.12.2, but never mind, the next one will be along soon enough.