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



/trunk/KDE/kdelibs/solid/solid/backends/udev/udevdvbinterface.h
<http://svn.reviewboard.kde.org/r/5993/#comment9776>

    Not really, as you started this source ;)
    It misses Christoph Pfister's copyright for 2007, so I will add that and 
then copy+add myself (for the very small changes I do...)



/trunk/KDE/kdelibs/solid/solid/backends/udev/udevdvbinterface.h
<http://svn.reviewboard.kde.org/r/5993/#comment9777>

    Right.


I will do the two changes above and then commit.

- Pino


On 2010-11-28 11:08:45, Pino Toscano wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5993/
> -----------------------------------------------------------
> 
> (Updated 2010-11-28 11:08:45)
> 
> 
> Review request for Solid.
> 
> 
> Summary
> -------
> 
> This is a first implementation of DvbInterface for the udev backend.
> 
> Allows devices of subsystem "dvb" among the ones that the udev backend shows, 
> and for them it provides a DvbInterface implementation (which is basically 
> copied from the HAL one, with s/hal/udev/ and the right property to query for 
> the device name, so I will do svn cp for it).
> What is missing is an icon and a description for dvb devices, mostly because 
> of the string freeze. (At least to me, HAL returns a "DVB Device" string for 
> all the dvb devices reported, so in case it would be the same for the udev 
> backend.)
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/solid/solid/CMakeLists.txt 1201550 
>   /trunk/KDE/kdelibs/solid/solid/backends/udev/udevdevice.cpp 1201550 
>   /trunk/KDE/kdelibs/solid/solid/backends/udev/udevdvbinterface.h 
> PRE-CREATION 
>   /trunk/KDE/kdelibs/solid/solid/backends/udev/udevdvbinterface.cpp 
> PRE-CREATION 
>   /trunk/KDE/kdelibs/solid/solid/backends/udev/udevmanager.cpp 1201550 
> 
> Diff: http://svn.reviewboard.kde.org/r/5993/diff
> 
> 
> Testing
> -------
> 
> Kaffeine sees and recognizes my DVB device, allowing me to use it.
> 
> 
> Thanks,
> 
> Pino
> 
>

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

Reply via email to