[Kde-hardware-devel] Review Request: Add (limited) Portable Media Player support to Solid udev backend

[Alex Merry]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5853/
-----------------------------------------------------------

Review request for Solid.


Summary
-------

Adds support to the Solid udev backend for Portable Media Players.

There are several FIXMEs in the code, all related to libmtp.  Basically, MTP is not supported at all.  In hal, support is provided by libmtp installing an FDI file that gets merged with the standard 10-usb-music-players.fdi.  All the relevant details are in one place, everyone is happy.

In this brave new world, the information that used to be shipped with hal (actually, with hal-info) about media players is now in the media-player-info package.  That doesn't include any details about MTP access, however.  libmtp ships some udev rules, but these just create a symlink (/dev/libmtp-%n) and don't provide us with anything useful.

Note that MTP devices are likely to be picked up by libgphoto2, however, and will be ID_MEDIA_PLAYER set to 1.  However, there is no way to distinguish these from random cameras.  Also, if a single USB device can operate as both an MTP device and a storage device (and these modes are not distinguished by a model id, for example), libgphoto2's udev rules can potentially overwrite the (useful) value of ID_MEDIA_PLAYER set by the media-player-info rules.

Basically, it's a big fat mess.


This addresses bug 253671.
    https://bugs.kde.org/show_bug.cgi?id=253671


Diffs
-----

  /trunk/KDE/kdelibs/solid/solid/CMakeLists.txt 1196931 
  /trunk/KDE/kdelibs/solid/solid/backends/udev/udevdevice.cpp 1196931 
  /trunk/KDE/kdelibs/solid/solid/backends/udev/udevmanager.cpp 1196931 
  /trunk/KDE/kdelibs/solid/solid/backends/udev/udevportablemediaplayer.h 1196931 
  /trunk/KDE/kdelibs/solid/solid/backends/udev/udevportablemediaplayer.cpp 1196931 
  /trunk/KDE/kdelibs/solid/solid/xdgbasedirs.cpp PRE-CREATION 
  /trunk/KDE/kdelibs/solid/solid/xdgbasedirs_p.h PRE-CREATION 

Diff: http://svn.reviewboard.kde.org/r/5853/diff


Testing
-------

solid-hardware lists "storage" as a supported protocol for my Sansa Clip+ when it's operating in MSC (storage) mode.


Thanks,

Alex

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20101115/021858bd/attachment.htm