]> git.ipfire.org Git - thirdparty/dbus.git/commitdiff
build: Make --enable-relocation compatible with hard-coding directories
authorSimon McVittie <smcv@collabora.com>
Wed, 29 Aug 2018 13:08:40 +0000 (14:08 +0100)
committerSimon McVittie <smcv@collabora.com>
Thu, 30 Aug 2018 16:44:20 +0000 (17:44 +0100)
Open Build Service RPMs for mingw32-dbus-1 hard-code all the
directories to make everything explicit, notably:

--prefix=/usr/i686-w64-mingw32/sys-root/mingw
--exec-prefix=/usr/i686-w64-mingw32/sys-root/mingw
...
--libdir=/usr/i686-w64-mingw32/sys-root/mingw/lib

Previously we didn't accept this as relocatable, but actually it's
fine: ${prefix} is still equivalent to ${libdir}/pkgconfig/../..,
so our relocation setup can work. Accept the result of expanding
"${prefix}" as an acceptable value for --exec-prefix, and accept the
results of expanding "${exec_prefix}/lib" etc. as acceptable values
for --libdir.

Note the use of single vs. double quotes here. A case statement that
matches '${prefix}' tests for the literal string «${prefix}»,
whereas a case that matches "${prefix}" tests for the string that is
the value of the variable named «prefix» that is set by the
--prefix command-line argument.

Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Philip Withnall <withnall@endlessm.com>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=107662

configure.ac

index f0d8358929d3ef9b833610086473959939628cc1..36c63b1a8ea0368946dda92f00844c6ed5bb6c39 100644 (file)
@@ -1578,7 +1578,7 @@ AC_ARG_ENABLE([relocation],
 can_relocate=yes
 
 AS_CASE(["${exec_prefix}"],
-  ['NONE'|'${prefix}'],
+  ['NONE'|'${prefix}'|"${prefix}"],
     [:],
   [*],
     [
@@ -1589,7 +1589,7 @@ AS_CASE(["${exec_prefix}"],
     ])
 
 AS_CASE(["${libdir}"],
-  ['${prefix}/lib'|'${prefix}/lib64'|'${exec_prefix}/lib'|'${exec_prefix}/lib64'],
+  ['${prefix}/lib'|'${prefix}/lib64'|'${exec_prefix}/lib'|'${exec_prefix}/lib64'|"${prefix}/lib"|"${exec_prefix}/lib"|"${prefix}/lib64"|"${exec_prefix}/lib64"],
     [:],
   [*],
     [