diff options
author | Xavier Claessens <xavier.claessens@collabora.com> | 2021-08-03 10:31:56 -0400 |
---|---|---|
committer | Xavier Claessens <xclaesse@gmail.com> | 2021-08-04 09:43:41 -0400 |
commit | 9da99e7a59cafdb9746ccd0cc148e454f0fa8b44 (patch) | |
tree | 76973679da1c9aea0780115a73209de168fbf6e8 | |
parent | 630a41eb815e2638b741cdbee42d9bb8509cd0a8 (diff) | |
download | meson-9da99e7a59cafdb9746ccd0cc148e454f0fa8b44.zip meson-9da99e7a59cafdb9746ccd0cc148e454f0fa8b44.tar.gz meson-9da99e7a59cafdb9746ccd0cc148e454f0fa8b44.tar.bz2 |
destdir: Allow relative to build directory
Meson already works like that, except in do_copydir() that requires
absolute destdir. Better explicitly support that instead of leaving it
undefined and unconsistent.
-rw-r--r-- | docs/markdown/Commands.md | 3 | ||||
-rw-r--r-- | docs/markdown/Installing.md | 6 | ||||
-rw-r--r-- | docs/markdown/snippets/relative_destdir.md | 6 | ||||
-rw-r--r-- | mesonbuild/minstall.py | 15 | ||||
-rw-r--r-- | unittests/allplatformstests.py | 6 |
5 files changed, 31 insertions, 5 deletions
diff --git a/docs/markdown/Commands.md b/docs/markdown/Commands.md index 3f3cf9f..ea68cc8 100644 --- a/docs/markdown/Commands.md +++ b/docs/markdown/Commands.md @@ -190,6 +190,9 @@ Install project to `$DESTDIR/prefix`: DESTDIR=/path/to/staging/area meson install -C builddir ``` +Since *0.60.0* `DESTDIR` and `--destdir` can be a path relative to build +directory. An absolute path will be set into environment when executing scripts. + ### rewrite *(since 0.50.0)* diff --git a/docs/markdown/Installing.md b/docs/markdown/Installing.md index 0e5cb12..42c71f1 100644 --- a/docs/markdown/Installing.md +++ b/docs/markdown/Installing.md @@ -113,6 +113,12 @@ is used just like with other build systems: $ DESTDIR=/path/to/staging/area meson install ``` +Since *0.57.0* `--destdir` argument can be used instead of environment. In that +case Meson will set `DESTDIR` into environment when runing install scripts. + +Since *0.60.0* `DESTDIR` and `--destdir` can be a path relative to build +directory. An absolute path will be set into environment when executing scripts. + ## Custom install behaviour Installation behaviour can be further customized using additional diff --git a/docs/markdown/snippets/relative_destdir.md b/docs/markdown/snippets/relative_destdir.md new file mode 100644 index 0000000..1498646 --- /dev/null +++ b/docs/markdown/snippets/relative_destdir.md @@ -0,0 +1,6 @@ +## Install DESTDIR relative to build directory + +When `DESTDIR` environment or `meson install --destdir` option is a relative path, +it is now assumed to be relative to the build directory. An absolute path will be +set into environment when executing scripts. It was undefined behavior in prior +Meson versions but was working as relative to build directory most of the time. diff --git a/mesonbuild/minstall.py b/mesonbuild/minstall.py index e753d94..654b547 100644 --- a/mesonbuild/minstall.py +++ b/mesonbuild/minstall.py @@ -494,11 +494,16 @@ class Installer: with open(datafilename, 'rb') as ifile: d = self.check_installdata(pickle.load(ifile)) - # Override in the env because some scripts could be relying on it. - if self.options.destdir is not None: - os.environ['DESTDIR'] = self.options.destdir - - destdir = os.environ.get('DESTDIR', '') + destdir = self.options.destdir + if destdir is None: + destdir = os.environ.get('DESTDIR') + if destdir and not os.path.isabs(destdir): + destdir = os.path.join(d.build_dir, destdir) + # Override in the env because some scripts could use it and require an + # absolute path. + if destdir is not None: + os.environ['DESTDIR'] = destdir + destdir = destdir or '' fullprefix = destdir_join(destdir, d.prefix) if d.install_umask != 'preserve': diff --git a/unittests/allplatformstests.py b/unittests/allplatformstests.py index aa8a3ee..a94d9cb 100644 --- a/unittests/allplatformstests.py +++ b/unittests/allplatformstests.py @@ -512,6 +512,12 @@ class AllPlatformTests(BasePlatformTests): self.assertEqual(logged, read_logs()) self.assertFalse(os.path.exists(self.installdir)) + # If destdir is relative to build directory it should install + # exactly the same files. + rel_installpath = os.path.relpath(self.installdir, self.builddir) + self._run(self.meson_command + ['install', '--dry-run', '--destdir', rel_installpath, '-C', self.builddir]) + self.assertEqual(logged, read_logs()) + def test_uninstall(self): exename = os.path.join(self.installdir, 'usr/bin/prog' + exe_suffix) dirname = os.path.join(self.installdir, 'usr/share/dir') |