Re: [PATCH 1/1] qga-vss: Use dynamic linking for GLib

[Paolo Bonzini <pbonzini@redhat.com>]

On 01/03/21 13:10, Marc-André Lureau wrote:
> If I read the old Makefile right, it's a bug we introduced by commit 
> 7272fc7 (meson: convert vss-win32), since glib was in the -shared list.
> 
> So it's legitimate to revert it to me:
> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Queued, thanks.

> 
>      >
>      > Signed-off-by: Kostiantyn Kostiuk <konstantin@daynix.com
>     <mailto:konstantin@daynix.com>>
>      > ---
>      >  qga/vss-win32/meson.build | 4 ++--
>      >  1 file changed, 2 insertions(+), 2 deletions(-)
>      >
>      > diff --git a/qga/vss-win32/meson.build b/qga/vss-win32/meson.build
>      > index 780c461432..90825edef3 100644
>      > --- a/qga/vss-win32/meson.build
>      > +++ b/qga/vss-win32/meson.build
>      > @@ -1,5 +1,5 @@
>      >  if add_languages('cpp', required: false)
>      > -  glib_static = dependency('glib-2.0', static: true)
>      > +  glib_dynamic = dependency('glib-2.0', static: false)
>      >    link_args =
>     cc.get_supported_link_arguments(['-fstack-protector-all',
>     '-fstack-protector-strong',
>      >                                               
>       '-Wl,--add-stdcall-alias', '-Wl,--enable-stdcall-fixup'])
>      >
>      > @@ -8,7 +8,7 @@ if add_languages('cpp', required: false)
>      >                  cpp_args: ['-Wno-unknown-pragmas',
>     '-Wno-delete-non-virtual-dtor', '-Wno-non-virtual-dtor'],
>      >                  link_args: link_args,
>      >                  vs_module_defs: 'qga-vss.def',
>      > -                dependencies: [glib_static, socket,
>      > +                dependencies: [glib_dynamic, socket,
>      >                                 cc.find_library('ole32'),
>      >                                 cc.find_library('oleaut32'),
>      >                                 cc.find_library('shlwapi'),
>      >
> 
> 
> 
> 
> -- 
> Marc-André Lureau