meson: fix seatdpath with absolute bindir v1 APPLIED

Alyssa Ross: 1
 meson: fix seatdpath with absolute bindir

 1 files changed, 1 insertions(+), 1 deletions(-)
Applied, thanks!
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~kennylevinsen/seatd-devel/patches/39633/mbox | git am -3
Learn more about email & git

[PATCH] meson: fix seatdpath with absolute bindir Export this patch

Quoting the Meson documentation:

> Note that the value returned for built-in options that end in `dir`
> such as `bindir` and `libdir` is usually a path relative to (and
> inside) the `prefix` but you should not rely on that, as it can also
> be an absolute path [in some cases](Builtin-options.md#universal-options).
> [`install_dir` arguments](Installing.md) handle that as expected
> but if you need an absolute path, e.g. to use in a define etc.,
> you should use the path concatenation operator like this:
> `get_option('prefix') / get_option('localstatedir')`.
> Never manually join paths as if they were strings.

The concatenation of two absolute paths caused seatd-launch in Nixpkgs
to try to launch e.g.
 meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meson.build b/meson.build
index 661b39a..0c6c823 100644
--- a/meson.build
+++ b/meson.build
@@ -22,7 +22,7 @@ if defaultpath == ''

seatdpath = '@0@/@1@/seatd'.format(get_option('prefix'), get_option('bindir'))
seatdpath = get_option('prefix') / get_option('bindir') / 'seatd'

cc = meson.get_compiler('c')