Review Request 124275: Allow building kcminput without libusb-compat

Aleix Pol Gonzalez aleixpol at kde.org
Wed Jul 8 11:28:07 UTC 2015



> On July 7, 2015, 1:06 a.m., Aleix Pol Gonzalez wrote:
> > Looks good to me too, as long as it works.
> > OTOH, it looks like a bad reason to offer an homogeneous experience, albeit being a not really interesting bit of Plasma.
> 
> Hrvoje Senjan wrote:
>     I submited this patch to upstream due to two reasons:
>     1) number of supported hardware & features is limited at best (according to link i posted, newest supported mouse is from 2004)
>     2) code is already #ifdef'ed, there is a small step to also let the buildsystem know the same
> 
> Aleix Pol Gonzalez wrote:
>     Oh no, don't get me wrong. Sending the patch upstream is the correct thing to do, or even better: support KDE dependencies on each distribution. But this can be just fine, in this case. If nobody gives you a "ship it" tomorrow, just go with it and push the change.
> 
> Martin Gräßlin wrote:
>     > newest supported mouse is from 2004
>     
>     Somehow that sounds like we should remove the code completely...
> 
> Kai Uwe Broulik wrote:
>     > Somehow that sounds like we should remove the code completely...
>     
>     +1

Well, if that's the case, let's merge this patch and remove the code altogether in another iteration.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124275/#review82157
-----------------------------------------------------------


On July 6, 2015, 8:41 p.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124275/
> -----------------------------------------------------------
> 
> (Updated July 6, 2015, 8:41 p.m.)
> 
> 
> Review request for Plasma, Aleix Pol Gonzalez, David Edmundson, and Martin Gräßlin.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Some distros are slowly targetting to get rid of the old libusb (see discussion in http://lists.opensuse.org/opensuse-factory/2015-07/msg00158.html)
> As the code mouse.[cpp,h] and logitechmouse.[cpp,h] is already ready for such an option, allow cmake to recognize this also.
> 
> 
> Diffs
> -----
> 
>   kcms/CMakeLists.txt d8a69a3 
>   kcms/input/CMakeLists.txt ee5c9c3 
> 
> Diff: https://git.reviewboard.kde.org/r/124275/diff/
> 
> 
> Testing
> -------
> 
> Builds.
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150708/d1b49224/attachment-0001.html>


More information about the Plasma-devel mailing list