GNOME Bugzilla – Bug 593933
Get rid of HAL
Last modified: 2010-05-06 14:28:41 UTC
Hal is deprecated, see http://www.freedesktop.org/wiki/Software/hal g-p-m uses DeviceKit-power now, but backlight handling still require hal when the backlight is not a xrandr device. All the X drivers need to support XBACKLIGHT before we can turn it off completely. Intel do, nouveau sortof does, so we could disable HAL in g-p-m using a #ifdef if we're sure the drivers support backlight
Created attachment 142374 [details] [review] Make HAL support optional Added new autogen.sh option: --enable-hal=[yes|no|auto] In auto mode It adds HAL support only if HAL development packages are installed. By default HAL support is enabled. This is one of my first autotools patches, so comments/corrections are more than welcome ;) Regards
Created attachment 142387 [details] [review] what i've put into git master I've changed your patch slightly, and added all the defines to the Makefiles and in the source. I've included it here for reference. Thanks!
Reopened until the code is completely removed
Javier / Richard: What *exactly* is still needed in this report to keep it opened?
All the major display drivers need XRANDR brightness support. Intel, nouveau and radeon should all work okay now, but there are a significant number of users of other cards that rely on HAL.
What about most Nvidia users, which intend to use the proprietary driver cause of it's 3d support.
commit 02f23be68cac1e5a01dddde152b6dd8de8c3e71a Author: Richard Hughes <richard@hughsie.com> Date: Thu Apr 29 14:47:26 2010 +0100 Remove the use of HAL from gnome-power-manager. Using HAL as a fallback has gone on long enough. People just need to be using a sane x drivers now