[PATCH] *BSD patches plus dbus-launch fix for regression tests
Havoc Pennington
hp at redhat.com
Tue Oct 24 17:59:34 PDT 2006
Hi,
Does this add -lpthread in the .pc file on Linux? That's something I
believe we're supposed to avoid, though hell if I understand what the
deal with libpthread is.
The dbus-launch change is only partly DBUS_BUILD_TESTS conditionalized.
I think a less intrusive patch would use an env variable, then you can
skip all the arg parsing etc. - in fact there may already be some
relevant env variable like the build dir set in a test context? What if
you just export the DBUS_TOP_BUILDDIR or something if isn't, from the sh
script calling dbus-launch?
You could also maybe just fall back to execlp #ifdef DBUS_BUILD_TESTS,
something I think we do elsewhere, then set the path in the script.
The HAVE_CMSGCRED change isn't everything that I meant; what I was
suggesting is that we add a HAVE_LOCAL_CREDS, and adding a configure
check to define that. And do configure in such a way that only one of
the two will ever be defined. The patch you have looks good fixing the
existing code, but the configure change would keep us from doing it
wrong in the future.
Thanks for subjecting yourself to freebsd for a noble cause ;-)
Havoc
More information about the dbus
mailing list