> On Sunday 11 of August 2013 23:11:40 François Andriot wrote: >> Le 11/08/2013 21:11, Slávek Banko a écrit : >> > Hello, >> > >> > A few days ago I sent a proposal to add a HAL backend for power >> > management. Meanwhile, I fixed call Suspend and added support for >> > governor (not tested). >> > >> > I also noticed that setting the brightness and governor is in >> condition >> > WITH_UPOWER, although in fact uses Trinity DBus daemon => has nothing >> to >> > do with upower. I added to the option WITH_TDEHWLIB_DAEMONS and fixed >> > incorrect WITH_UPOWER conditions. >> > >> > Please test it. At the same time I'm waiting for your opinions - >> > incorporate it or not? >> > >> > Thanks >> > Slavek >> >> Hello, this patch is good news :-) >> I suggest that HAL support is disabled by default (OFF) instead of >> WITH_ALL_OPTIONS. >> >> Appart from that, I have troubles building tdelibs R14 on CentOS 5, >> because UDEV is too old (095). >> I've also noticed that I can NOT build tdelibs without tdehwlib at all >> (no configuration option) ... >> That means, in the current state, CentOS 5 will be limited to TDE >> 3.5.13-SRU branch... >> >> Francois >> > > The advantage of this patch is that it does not add any specific > dependence > due to HAL. Only adds a few dbus calls. So there is no problem when it is > hal > enabled, but not installed. > > Problems with the old version of udev can be solved? > > Slavek > -- I am in agreement that the HAL support should default to OFF. DBUS calls are slow enough as it is; we don't need calls to extraneous services slowing things down even more. :-) Tim