Attachment 47888 Details for Bug 34405 – [PATCH 3/5 v2] Run dbus-test from test/, not dbus/

[patch] [PATCH 3/5 v2] Run dbus-test from test/, not dbus/

0001-Run-dbus-test-from-test-not-dbus.patch (text/plain), 2.20 KB, created by Simon McVittie on 2011-06-13 03:10:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simon McVittie

Created: 2011-06-13 03:10:24 UTC

Size: 2.20 KB

patch

obsolete

>From 9844adaf55b7467d64a952519213b5c0813a9d63 Mon Sep 17 00:00:00 2001
>From: Simon McVittie <simon.mcvittie@collabora.co.uk>
>Date: Mon, 13 Jun 2011 11:08:28 +0100
>Subject: [PATCH] Run dbus-test from test/, not dbus/
>
>This avoids failures caused by test/data not having been set up yet.
>
>Bug: https://bugs.freedesktop.org/show_bug.cgi?id=34405
>---
> dbus/Makefile.am |   15 ++++-----------
> test/Makefile.am |   11 ++++++++++-
> 2 files changed, 14 insertions(+), 12 deletions(-)
>
>diff --git a/dbus/Makefile.am b/dbus/Makefile.am
>index 2e6ca1d..094773c 100644
>--- a/dbus/Makefile.am
>+++ b/dbus/Makefile.am
>@@ -269,20 +269,13 @@ libdbus_internal_la_CPPFLAGS = -DDBUS_STATIC_BUILD
> libdbus_internal_la_LIBADD=$(DBUS_CLIENT_LIBS)
> libdbus_internal_la_LDFLAGS=$(export_symbols_internal) @R_DYNAMIC_LDFLAG@
> 
>-## note that TESTS has special meaning (stuff to use in make check)
>-## so if adding tests not to be run in make check, don't add them to
>-## TESTS
>+noinst_PROGRAMS =
>+
> if DBUS_BUILD_TESTS
>-TESTS_ENVIRONMENT=DBUS_TEST_DATA=$(top_builddir)/test/data DBUS_TEST_HOMEDIR=$(top_builddir)/dbus
>-TESTS=dbus-test
>-else
>-TESTS=
>+# We can't actually run this til we've reached test/
>+noinst_PROGRAMS += dbus-test
> endif
> 
>-## we use noinst_PROGRAMS not check_PROGRAMS so that we build
>-## even when not doing "make check"
>-noinst_PROGRAMS=$(TESTS)
>-
> dbus_test_SOURCES=				\
> 	dbus-test-main.c
> 
>diff --git a/test/Makefile.am b/test/Makefile.am
>index d537321..564300d 100644
>--- a/test/Makefile.am
>+++ b/test/Makefile.am
>@@ -30,6 +30,13 @@ TESTS = \
> 	shell-test \
> 	$(NULL)
> 
>+## These are conceptually part of directories that come earlier in SUBDIRS
>+## order, but we don't want to run them til we arrive in this directory,
>+## since they depend on stuff from this directory
>+TESTS += \
>+	../dbus/dbus-test \
>+	$(NULL)
>+
> else !DBUS_BUILD_TESTS
> 
> TEST_BINARIES=
>@@ -105,7 +112,9 @@ installcheck_environment = \
> 
> TESTS_ENVIRONMENT = \
> 	DBUS_TEST_DAEMON=@abs_top_builddir@/bus/dbus-daemon$(EXEEXT) \
>-	DBUS_TEST_DATA=@abs_top_builddir@/test/data
>+	DBUS_TEST_DATA=@abs_top_builddir@/test/data \
>+	DBUS_TEST_HOMEDIR=@abs_top_builddir@/dbus \
>+	$(NULL)
> 
> test_corrupt_SOURCES = corrupt.c
> test_corrupt_CPPFLAGS = $(GLIB_CFLAGS) $(DBUS_GLIB_CFLAGS)
>-- 
>1.7.5.4
>

Actions: View | Splinter Review

Attachments on bug 34405: 47609 | 47610 | 47611 | 47612 | 47613 | 47888 | 47890 | 47993

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.