On Aug 13, 2012 2:40 AM, "Timothy Pearson" <kb9vqf(a)pearsoncomputing.net
wrote:
>>> > Seems then that my temporary patch is best? Just delete
>>> the if( WITH_HAL ) test and build with dbus-tqt support regardless of
>>> how
>>> WITH_HAL or WITH_TDEHWLIB are set?
>>
>>> I would
check to see if either WITH_HAL or WITH_TDEHWLIB are
>>> set before requiring dbus-tqt; if neither are set then tdebase does
not
>>> depend on it.
>
>> I ran a tdebase build with
WITH_HAL=NO, WITH_TDEHWLIB=NO and no related
>> patching. The build fails with the same error about not finding
>> dbus/dbus.h. The failure occurs when building tdm backend, in
>> consolekit.c.
> Sounds like the code in consolekit.c is
not properly guarded with
#ifdefs,
which would be
set when dbus-tqt is found.
Tim
Is there any good reason that dbus might not be available on a target
distribution/system? If not, I would say to just make dbus a tdebase
dependency and remove the if statements from around the dbus checks. If
there is a good reason for dbus to be missing, then we will need to add
another flag to build without dbus support and also add the requisite
#ifdefs to the tdm consolekit code.
Thoughts?
Tim
---------------------------------------------------------------------
To unsubscribe, e-mail:
trinity-devel-unsubscribe(a)lists.pearsoncomputing.net
For additional commands, e-mail:
trinity-devel-help(a)lists.pearsoncomputing.net
Read list messages on the web archive:
http://trinity-devel.pearsoncomputing.net/
Please remember not to top-post:
http://trinity.pearsoncomputing.net/mailing_lists/#top-posting
We should assume all users have dbus just like dcop