After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 603470 - Do not abort if hal is not available
Do not abort if hal is not available
Status: RESOLVED FIXED
Product: gnome-power-manager
Classification: Deprecated
Component: gnome-power-manager
SVN TRUNK
Other Linux
: Normal normal
: ---
Assigned To: GNOME Power Manager Maintainer(s)
GNOME Power Manager Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-12-01 11:31 UTC by Martin Pitt
Modified: 2010-01-18 10:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
do not abort if hal is not installed (903 bytes, patch)
2009-12-01 11:31 UTC, Martin Pitt
none Details | Review
do not connect to hal if xrandr is available (1.60 KB, patch)
2010-01-14 17:21 UTC, Martin Pitt
accepted-commit_now Details | Review

Description Martin Pitt 2009-12-01 11:31:02 UTC
Created attachment 148816 [details] [review]
do not abort if hal is not installed

g-p-m now runs sufficiently well without hal, so people who have working
XBACKLIGHT (or don't care about missing notifications) should be able to
uninstall hal.

However, if you do this, g-p-m exits immediately.

With attached patch, I can run g-p-m on a hal-less GNOME desktop.
Comment 1 Richard Hughes 2009-12-01 13:00:44 UTC
I'm tempted to remove all the libhal-glib stuff, and remove the HAL brightness changing functionality completely. Or is that too drastic?
Comment 2 Martin Pitt 2009-12-01 14:03:04 UTC
My gut feeling is that it's still a little too early for that, since xbacklight does not yet work on many hardware platforms. However, you probably have a better insight into which drivers would fall out of the "works" spectrum with this.
Comment 3 Martin Pitt 2010-01-14 17:21:55 UTC
Created attachment 151414 [details] [review]
do not connect to hal if xrandr is available

Nowadays it is a valid setup to have hal installed, but only start on demand
via D-Bus activation (instead of during boot). If we detect xrandr-capable
hardware, do not try to connect to hal, to avoid triggering it.

I tested this on a Dell Mini 10 (which does have XBACKLIGHT support), where hal is not triggered any more. I also tested it on my Dell Latitude D430, which doesn't have xrandr backlight support right now, and there hal is properly D-Bus activated and g-p-m backlight control/notification feedback works.

Thanks for considering!

Martin
Comment 4 Richard Hughes 2010-01-14 18:23:20 UTC
Review of attachment 151414 [details] [review]:

looks good, please commit.
Comment 5 Martin Pitt 2010-01-18 10:13:13 UTC
Sebastien Bacher committed the patches for me, thanks for review!