[Kde-hardware-devel] Review Request 102983: Added support for rfkill

[David Rosca]

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


Can you please discard this review? libbluedevil is not used in recent Plasma release (replaced with bluez-qt which have support for rfkill), and there won't be any more KDE4 nor Plasma 5.2.x releases.

- David Rosca


On July 11, 2015, 8:43 p.m., Rüdiger Sonderfeld wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/102983/
> -----------------------------------------------------------
> 
> (Updated July 11, 2015, 8:43 p.m.)
> 
> 
> Review request for Plasma and Solid.
> 
> 
> Repository: libbluedevil
> 
> 
> Description
> -------
> 
> rfkill is a subsystem of the Linux Kernel to disable radio transmitters (including Bluetooth). This patch adds support for rfkill to libbluedevil. An interface is added to the Manager class.
> 
> rfkill support is important because gnome uses rfkill to deactivate bluetooth. Therefore if bluetooth is currently disabled in the gnome-control-center it can not be used by kde. With this patch the proper API to reenable bluetooth is provided.
> 
> Disabling bluetooth with rfkill has the advantage that some computers switch off the bluetooth LED. Which does not happen (at least on my Thinkpad) if it's just powered off (as kde does at the moment).
> 
> rfkill can also detect if bluetooth has been deactivated by a hardware switch.
> 
> btw. please mark https://gitorious.org/libbluedevil/libbluedevil as obsolete and update http://projects.ufocoders.com/projects/libbluedevil/wiki. I used the old gitorious code base at first.
> 
> 
> Diffs
> -----
> 
>   bluedevil/bluedevilmanager.h 0386b8e 
>   bluedevil/bluedevilmanager.cpp 594d3bd 
>   bluedevil/test/bluedeviltest.cpp 2d3d0e2 
> 
> Diff: https://git.reviewboard.kde.org/r/102983/diff/
> 
> 
> Testing
> -------
> 
> I added a test to bluedeviltest and it works for me.
> 
> 
> Thanks,
> 
> Rüdiger Sonderfeld
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20150712/be5ec423/attachment.html>