[PATCH] *BSD patches plus dbus-launch fix for regression tests

John (J5) Palmieri johnp at redhat.com
Wed Oct 25 16:53:56 PDT 2006

Forgot the patch.

On Wed, 2006-10-25 at 19:53 -0400, John (J5) Palmieri wrote:
> On Tue, 2006-10-24 at 20:59 -0400, Havoc Pennington wrote:
> > Hi,
> > 
> > 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?
> Attached is a fixed up patch that just has config.in define the binaries
> location.
> > 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.
> I gave up on this.  A real *BSD hacker needs to look at this since there
> are numerous code paths.  I think some of them are the same but I can't
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dbus-launch-exec-local-bus-for-tests2.patch
Type: text/x-patch
Size: 2042 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/dbus/attachments/20061025/bd5ee684/dbus-launch-exec-local-bus-for-tests2.bin

More information about the dbus mailing list