diff options
| author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2024-03-19 13:25:02 +0400 |
|---|---|---|
| committer | Michael Tokarev <mjt@tls.msk.ru> | 2024-03-21 20:13:44 +0300 |
| commit | f8fb5928a850f889bbb949486149a5de3da8295b (patch) | |
| tree | 9269cd80d8f7b6f6fe2e45d15282877daf57cde1 /ui | |
| parent | de742b16b3253bb9a194d623d0b751844bed8bc7 (diff) | |
ui: compile dbus-display1.c with -fPIC as necessary
Building dbus-display1.c explicitly as a static library drops -fPIC by
default, which may not be correct if it ends up linked to a shared
library.
Let the target decide how to build the unit, with or without -fPIC. This
makes commit 186acfbaf7 ("tests/qtest: Depend on dbus_display1_dep") no
longer relevant, as dbus-display1.c will be recompiled.
Fixes: c172136ea33 ("meson: ensure dbus-display generated code is built
before other units")
Reported-by: Olaf Hering <olaf@aepfle.de>
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
(cherry picked from commit d4069a84a3380247c1b524096c6a807743bf687a)
Diffstat (limited to 'ui')
| -rw-r--r-- | ui/meson.build | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ui/meson.build b/ui/meson.build index 0f09d31c60..27e2e0ca6c 100644 --- a/ui/meson.build +++ b/ui/meson.build @@ -91,8 +91,7 @@ if dbus_display '--interface-prefix', 'org.qemu.', '--c-namespace', 'QemuDBus', '--generate-c-code', '@BASENAME@']) - dbus_display1_lib = static_library('dbus-display1', dbus_display1, dependencies: gio) - dbus_display1_dep = declare_dependency(link_with: dbus_display1_lib, sources: dbus_display1[0]) + dbus_display1_dep = declare_dependency(sources: dbus_display1, dependencies: gio) dbus_ss.add(when: [gio, dbus_display1_dep], if_true: [files( 'dbus-chardev.c', |