Review Request: Fix "smb" and "nfs" remote collections to work with Dynamic Collections

Peter C. Ndikuwera pndiku at gmail.com
Fri Feb 25 15:43:03 CET 2011


Any comments on this, or can I commit?

"Happiness makes up in height what it lacks in length"
-- Robert Frost


On 23 February 2011 22:34, Peter C. Ndikuwera <pndiku at gmail.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100725/
>   Review request for Amarok.
> By Peter C. Ndikuwera.
> Description
>
> Used MassStorageDeviceHandler as a template, then changed the following:
>
> SQLStorage - should use StorageAccess instead of StorageVolume. StorageVolume only sees physical volumes on the local PC. StorageAccess sees all accessible storage.
>
> NFS & SMS DeviceHandlers use KMountPoint to find where the disk is mounted.
>
>   Testing
>
> Mounted nfs and samba/cifs shares and did full rescans. Then unmounted them one by one. Works perfectly.
>
> Haven't tested what happens if you mount a share that's _not_ in /etc/fstab.
>
>   Diffs
>
>    - src/core-impl/collections/db/sql/MountPointManager.cpp (ed29f48)
>    - src/core-impl/collections/db/sql/device/CMakeLists.txt (1baaf85)
>    - src/core-impl/collections/db/sql/device/nfs/CMakeLists.txt (6807dd6)
>    - src/core-impl/collections/db/sql/device/nfs/NfsDeviceHandler.h
>    (21c1894)
>    - src/core-impl/collections/db/sql/device/nfs/NfsDeviceHandler.cpp
>    (91af710)
>    - src/core-impl/collections/db/sql/device/smb/CMakeLists.txt (fc27d5a)
>    - src/core-impl/collections/db/sql/device/smb/SmbDeviceHandler.h
>    (c74c840)
>    - src/core-impl/collections/db/sql/device/smb/SmbDeviceHandler.cpp
>    (2f19e28)
>
> View Diff <http://git.reviewboard.kde.org/r/100725/diff/>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20110225/ca97f551/attachment.htm 


More information about the Amarok-devel mailing list