diff options
| author | Paolo Bonzini | 2021-08-11 12:05:50 +0200 |
|---|---|---|
| committer | Alex Bennée | 2021-09-02 12:29:34 +0200 |
| commit | b906acace2d4f68b6ff8de73739a773cc4851436 (patch) | |
| tree | 42e6cd9dda73a14c6d714e175c6e989b6ea0a7cd /configure | |
| parent | gitlab-ci: Fix ..._RUNNER_AVAILABLE variables and document them (diff) | |
| download | qemu-b906acace2d4f68b6ff8de73739a773cc4851436.tar.gz qemu-b906acace2d4f68b6ff8de73739a773cc4851436.tar.xz qemu-b906acace2d4f68b6ff8de73739a773cc4851436.zip | |
plugins: do not limit exported symbols if modules are active
On Mac --enable-modules and --enable-plugins are currently incompatible, because the
Apple -Wl,-exported_symbols_list command line options prevents the export of any
symbols needed by the modules. On x86 -Wl,--dynamic-list does not have this effect,
but only because the -Wl,--export-dynamic option provided by gmodule-2.0.pc overrides
it. On Apple there is no -Wl,--export-dynamic, because it is the default, and thus
no override.
Either way, when modules are active there is no reason to include the plugin_ldflags.
While at it, avoid the useless -Wl,--export-dynamic when --enable-plugins is
specified but --enable-modules is not; this way, the GNU and Apple configurations
are more similar.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/516
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
[AJB: fix noexport to no-export]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210811100550.54714-1-pbonzini@redhat.com>
Cc: qemu-stable@nongnu.org
Diffstat (limited to 'configure')
| -rwxr-xr-x | configure | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -3187,9 +3187,8 @@ glib_req_ver=2.56 glib_modules=gthread-2.0 if test "$modules" = yes; then glib_modules="$glib_modules gmodule-export-2.0" -fi -if test "$plugins" = "yes"; then - glib_modules="$glib_modules gmodule-2.0" +elif test "$plugins" = "yes"; then + glib_modules="$glib_modules gmodule-no-export-2.0" fi for i in $glib_modules; do |