Hi On Thu, Jun 1, 2017 at 5:08 PM Thomas Lamprecht <[email protected]> wrote:
> glib depends on libpcre which was not shipped with the MSI, thus > starting of the qemu-ga.exe failed with the respective error message. > > Tell WIXL to ship this library with the MSI to avoid this problem. > > Signed-off-by: Thomas Lamprecht <[email protected]> > CC: Stefan Weil <[email protected]> > CC: Michael Roth <[email protected]> > It depends on your glib build, but since Fedora is one of the most maintained cross mingw- distrib, it make sense to fix the build there. Other solutions would involve either using wixl-specific require preprocessor directive (which comes with a bunch of unused files since those are mostly generated from mingw*- packages), or coming up with some kind of dynamic dependency resolution (approach similar to Richard W.M. Jones nsiswrapper). However this last approach is quite limited, since it doesn't reach to data files etc. In the meantime: Reviewed-by: Marc-André Lureau <[email protected]> > I haven't done much with the qga or WIXL, so I send this as a RFC. > I hope that I guessed the right people to get CC'ed from MAINTAINERS. > > This fixes a current qemu-ga MSI build, I tested it successfully with > Windows 7 > and Windows 10 as guest OS. > > I cross built from a Fedora 25 LXC container. > > The Guid for the libpcre was generated by https://www.guidgen.com/ as > suggested > by: > > http://wixtoolset.org/documentation/manual/v3/howtos/general/generate_guids.html > > qga/installer/qemu-ga.wxs | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/qga/installer/qemu-ga.wxs b/qga/installer/qemu-ga.wxs > index fa2260cafa..5af11627f8 100644 > --- a/qga/installer/qemu-ga.wxs > +++ b/qga/installer/qemu-ga.wxs > @@ -125,6 +125,9 @@ > <Component Id="libwinpthread" > Guid="{6C117C78-0F47-4B07-8F34-6BEE11643829}"> > <File Id="libwinpthread_1.dll" Name="libwinpthread-1.dll" > Source="$(var.Mingw_bin)/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.Mingw_bin)/libpcre-1.dll" KeyPath="yes" DiskId="1"/> > + </Component> > <Component Id="registry_entries" > Guid="{D075D109-51CA-11E3-9F8B-000C29858960}"> > <RegistryKey Root="HKLM" > > Key="Software\$(env.QEMU_GA_MANUFACTURER)\$(env.QEMU_GA_DISTRO)\Tools\QemuGA"> > @@ -173,6 +176,7 @@ > <ComponentRef Id="libssp" /> > <ComponentRef Id="libwinpthread" /> > <ComponentRef Id="registry_entries" /> > + <ComponentRef Id="libpcre" /> > </Feature> > > <InstallExecuteSequence> > -- > 2.11.0 > > > > -- Marc-André Lureau
