Reviewed-by: Konstantin Kostiuk <kkost...@redhat.com> Tested-by: Konstantin Kostiuk <kkost...@redhat.com>
On Tue, Dec 13, 2022 at 5:13 PM Andrey Drobyshev < andrey.drobys...@virtuozzo.com> wrote: > According to GLib changelog [1], since version 2.73.2 GLib is using > libpcre2 instead of libpcre. As a result, qemu-ga MSI installation > fails due to missing DLL when linked with the newer GLib. > > This commit makes wixl to put the right libpcre version into the MSI > bundle: either libpcre-1.dll or libpcre2-8-0.dll, depending on the > present version of GLib. > > [1] https://gitlab.gnome.org/GNOME/glib/-/releases#2.73.2 > > Previous version: > https://lists.nongnu.org/archive/html/qemu-trivial/2022-11/msg00237.html > > Signed-off-by: Andrey Drobyshev <andrey.drobys...@virtuozzo.com> > --- > qga/installer/qemu-ga.wxs | 12 +++++++++--- > qga/meson.build | 6 ++++++ > 2 files changed, 15 insertions(+), 3 deletions(-) > > diff --git a/qga/installer/qemu-ga.wxs b/qga/installer/qemu-ga.wxs > index e344c38e74..9f0bacae81 100644 > --- a/qga/installer/qemu-ga.wxs > +++ b/qga/installer/qemu-ga.wxs > @@ -101,9 +101,15 @@ > <Component Id="libwinpthread" > Guid="{6C117C78-0F47-4B07-8F34-6BEE11643829}"> > <File Id="libwinpthread_1.dll" Name="libwinpthread-1.dll" > Source="$(var.BIN_DIR)/libwinpthread-1.dll" KeyPath="yes" DiskId="1"/> > </Component> > - <Component Id="libpcre" > Guid="{7A86B45E-A009-489A-A849-CE3BACF03CD0}"> > - <File Id="libpcre_1.dll" Name="libpcre-1.dll" > Source="$(var.BIN_DIR)/libpcre-1.dll" KeyPath="yes" DiskId="1"/> > - </Component> > + <?if $(var.LIBPCRE) = "libpcre1"?> > + <Component Id="libpcre" > Guid="{7A86B45E-A009-489A-A849-CE3BACF03CD0}"> > + <File Id="libpcre_1.dll" Name="libpcre-1.dll" > Source="$(var.BIN_DIR)/libpcre-1.dll" KeyPath="yes" DiskId="1"/> > + </Component> > + <?else?> > + <Component Id="libpcre" > Guid="{F92A3804-B59C-419D-8F29-99A30352C156}"> > + <File Id="libpcre2_8_0.dll" Name="libpcre2-8-0.dll" > Source="$(var.BIN_DIR)/libpcre2-8-0.dll" KeyPath="yes" DiskId="1"/> > + </Component> > + <?endif?> > <Component Id="registry_entries" > Guid="{D075D109-51CA-11E3-9F8B-000C29858960}"> > <RegistryKey Root="HKLM" > > Key="Software\$(var.QEMU_GA_MANUFACTURER)\$(var.QEMU_GA_DISTRO)\Tools\QemuGA"> > diff --git a/qga/meson.build b/qga/meson.build > index 1ff159edc1..ad17dc7dca 100644 > --- a/qga/meson.build > +++ b/qga/meson.build > @@ -140,6 +140,11 @@ if targetos == 'windows' > qemu_ga_msi_vss = ['-D', 'InstallVss'] > deps += qga_vss > endif > + if glib.version() < '2.73.2' > + libpcre = 'libpcre1' > + else > + libpcre = 'libpcre2' > + endif > qga_msi = custom_target('QGA MSI', > input: files('installer/qemu-ga.wxs'), > output: 'qemu-ga-@0@.msi'.format(host_arch), > @@ -153,6 +158,7 @@ if targetos == 'windows' > '-D', 'QEMU_GA_VERSION=' + > config_host['QEMU_GA_VERSION'], > '-D', 'QEMU_GA_MANUFACTURER=' + > config_host['QEMU_GA_MANUFACTURER'], > '-D', 'QEMU_GA_DISTRO=' + > config_host['QEMU_GA_DISTRO'], > + '-D', 'LIBPCRE=' + libpcre, > ]) > all_qga += [qga_msi] > alias_target('msi', qga_msi) > -- > 2.34.3 > >
