From: Simon McVittie <smcv@collabora.com>
Date: Wed, 17 Apr 2019 15:32:01 +0000 (+0100)
Subject: build: Don't assume we can set permissions on a directory
X-Git-Tag: dbus-1.13.10~6^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=14f46d14;p=thirdparty%2Fdbus.git

build: Don't assume we can set permissions on a directory

MSYS2 has enough of a Unixish environment to run Autotools, but
apparently not enough of a Unixish environment to have functional
permissions.

Closes: dbus#216
---

diff --git a/test/Makefile.am b/test/Makefile.am
index 27d19460e..559590b23 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -706,8 +706,11 @@ EXTRA_DIST += $(static_data)
 
 ## copy tests to builddir so that generated tests and static tests
 ## are all in one place.
+## Note that we can't create the XDG_RUNTIME_DIR with permissions 0700
+## on MSYS2, which rejects attempts to change permissions, hence "|| true".
 all-local: copy-config-local uninstalled-config-local
-	$(AM_V_at)$(MKDIR_P) -m 700 XDG_RUNTIME_DIR
+	$(AM_V_at)$(MKDIR_P) XDG_RUNTIME_DIR
+	$(AM_V_at)chmod 0700 XDG_RUNTIME_DIR || true
 
 copy-config-local:
 	$(AM_V_at)$(MKDIR_P) data/valid-config-files/session.d