[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 5/5] ui: compile dbus-display1.c with -fPIC as necessary
From: |
marcandre . lureau |
Subject: |
[PULL 5/5] ui: compile dbus-display1.c with -fPIC as necessary |
Date: |
Wed, 20 Mar 2024 17:53:49 +0400 |
From: Marc-André Lureau <marcandre.lureau@redhat.com>
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>
---
ui/meson.build | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/ui/meson.build b/ui/meson.build
index 0b7e2b6f6b..a5ce22a678 100644
--- a/ui/meson.build
+++ b/ui/meson.build
@@ -90,8 +90,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',
--
2.44.0
- [PULL 2/5] ui/dbus: factor out sending a scanout, (continued)
- [PULL 2/5] ui/dbus: factor out sending a scanout, marcandre . lureau, 2024/03/12
- [PULL 5/5] virtio-gpu: fix scanout migration post-load, marcandre . lureau, 2024/03/12
- [PULL 4/5] virtio-gpu: remove needless condition, marcandre . lureau, 2024/03/12
- [PULL 3/5] ui/dbus: filter out pending messages when scanout, marcandre . lureau, 2024/03/12
- Re: [PULL 0/5] UI patches, Peter Maydell, 2024/03/12
- [PULL 0/5] Ui patches, marcandre . lureau, 2024/03/20
- [PULL 1/5] ui/vc: Do not inherit the size of active console, marcandre . lureau, 2024/03/20
- [PULL 2/5] ui/vnc: Do not use console_select(), marcandre . lureau, 2024/03/20
- [PULL 3/5] ui/cocoa: Do not use console_select(), marcandre . lureau, 2024/03/20
- [PULL 4/5] ui/curses: Do not use console_select(), marcandre . lureau, 2024/03/20
- [PULL 5/5] ui: compile dbus-display1.c with -fPIC as necessary,
marcandre . lureau <=
- Re: [PULL 0/5] Ui patches, Peter Maydell, 2024/03/20