]> git.ipfire.org Git - thirdparty/dbus.git/commitdiff
bus: Update a build system cross-reference
authorSimon McVittie <smcv@collabora.com>
Tue, 10 Dec 2024 11:00:28 +0000 (11:00 +0000)
committerSimon McVittie <smcv@collabora.com>
Wed, 11 Dec 2024 13:13:30 +0000 (13:13 +0000)
We no longer have an Autotools build system, but Meson sets the same
environment variables that Autotools used to set. CMake does not, yet,
but ideally should.

Resolves: https://gitlab.freedesktop.org/dbus/dbus/-/issues/538
Signed-off-by: Simon McVittie <smcv@collabora.com>
(cherry picked from commit 229e48f549dfb1af891545fd978974dd462777da)

bus/config-parser.c
test/CMakeLists.txt

index 2b630d181f870a9a5cb879441a297dab740c5bf3..cb388473efaced78ac890bfc5a85b93a768ba2dd 100644 (file)
@@ -3815,7 +3815,7 @@ test_default_session_servicedirs (const DBusString *test_base_dir)
     {
       _dbus_test_diag ("Not testing default session service directories because a "
               "build-time testing environment variable is not set: "
-              "see AM_TESTS_ENVIRONMENT in tests/Makefile.am");
+              "see test_env in tests/meson.build");
       ret = TRUE;
       goto out;
     }
@@ -3835,7 +3835,7 @@ test_default_session_servicedirs (const DBusString *test_base_dir)
   if (!_dbus_string_append (&runtime_dir_based, "/dbus-1/services"))
     _dbus_test_fatal ("out of memory");
 
-  /* Sanity check: the Makefile sets this up. We assume that if this is
+  /* The Meson build system sets this up. We assume that if this is
    * right, the XDG_DATA_DIRS will be too. */
   if (!_dbus_string_starts_with_c_str (&data_home_based, dbus_test_builddir))
     _dbus_test_fatal ("$XDG_DATA_HOME should start with $DBUS_TEST_BUILDDIR");
index 032ba1f7eb7c9b5f8c00e559de1dc05514ee1a18..f3fd8d9cd0ec37377e99df3502d3c4b484911164 100644 (file)
@@ -149,6 +149,9 @@ if(DBUS_ENABLE_INTRUSIVE_TESTS)
 
     set(SOURCES bus/main.c bus/common.c bus/common.h)
     add_test_executable(test-bus "${SOURCES}" dbus-daemon-internal dbus-testutils ${EXPAT_LIBRARIES})
+    # TODO: For full coverage this should be run with some specific
+    # environment variable values: see test_env in tests/meson.build,
+    # and dbus#538
     set_target_properties(test-bus PROPERTIES COMPILE_FLAGS ${DBUS_INTERNAL_CLIENT_DEFINITIONS})
 
     if(ENABLE_TRADITIONAL_ACTIVATION)