-----------------------------------------------------------
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

_______________________________________________
Kde-hardware-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-hardware-devel

Reply via email to