aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndres Freund <andres@anarazel.de>2022-10-01 12:39:12 -0700
committerAndres Freund <andres@anarazel.de>2022-10-01 12:39:12 -0700
commitd0366bfb3b211b26a997a4220757392a0c153d8d (patch)
tree6dfe97fe310de90b8fadb234e21ca33d35c3356d
parenta9d58bfe8a3ae2254e1553ab76974feeaafa0133 (diff)
downloadpostgresql-d0366bfb3b211b26a997a4220757392a0c153d8d.tar.gz
postgresql-d0366bfb3b211b26a997a4220757392a0c153d8d.zip
meson: windows: Determine path to tmp_install + prefix using meson
Previously some paths (like c:\ or d:/) worked, but plenty others (like /path/to or //computer/share/path) didn't. As we'd like to change the default prefix to /usr/local/pgsql, that's a problem. Instead of trying to do this in meson.build, call out to the implementation meson install uses. This isn't pretty, but it's more reliable than what we had before. Discussion: https://postgr.es/CAEG8a3LGWE-gG6vuddmH91RORhi8gWs0mMB-hcTmP3_NVgM7dg@mail.gmail.com
-rw-r--r--meson.build40
1 files changed, 16 insertions, 24 deletions
diff --git a/meson.build b/meson.build
index c32cd792db8..c3bbb1c5fca 100644
--- a/meson.build
+++ b/meson.build
@@ -29,6 +29,7 @@ fs = import('fs')
pkgconfig = import('pkgconfig')
host_system = host_machine.system()
+build_system = build_machine.system()
host_cpu = host_machine.cpu_family()
cc = meson.get_compiler('c')
@@ -2748,32 +2749,23 @@ endif
# Test prep
###############################################################
-# The determination of where a DESTDIR install points to is ugly, it's somewhat hard
-# to combine two absolute paths portably...
-
-prefix = get_option('prefix')
-
-test_prefix = fs.as_posix(prefix)
-
-if fs.is_absolute(get_option('prefix'))
- if host_system == 'windows'
- if prefix.split(':/').length() == 1
- # just a drive
- test_prefix = ''
- else
- test_prefix = prefix.split(':/')[1]
- endif
- else
- assert(prefix.startswith('/'))
- test_prefix = './@0@'.format(prefix)
- endif
-endif
-
-# DESTDIR for the installation used to run tests in
+# DESTDIR for the installation we'll run tests in
test_install_destdir = meson.build_root() / 'tmp_install/'
-# DESTDIR + prefix appropriately munged
-test_install_location = test_install_destdir / test_prefix
+# DESTDIR + prefix appropriately munged
+if build_system != 'windows'
+ # On unixoid systems this is trivial, we just prepend the destdir
+ assert(dir_prefix.startswith('/')) # enforced by meson
+ test_install_location = '@0@@1@'.format(test_install_destdir, dir_prefix)
+else
+ # drives, drive-relative paths, etc make this complicated on windows, call
+ # meson's logic for it
+ command = [
+ meson_bin, meson_args, 'runpython', '-c',
+ 'import sys; from mesonbuild.scripts import destdir_join; print(destdir_join(sys.argv[4], sys.argv[5]))',
+ test_install_destdir, dir_prefix]
+ test_install_location = run_command(command, check: true).stdout().strip()
+endif
meson_install_args = meson_args + ['install'] + {
'meson': ['--quiet', '--only-changed', '--no-rebuild'],