[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 1/3] qga: Installer: Wait for installation to finish
From: |
Marc-André Lureau |
Subject: |
Re: [PATCH v3 1/3] qga: Installer: Wait for installation to finish |
Date: |
Tue, 24 Mar 2020 15:00:35 +0100 |
Hi
On Tue, Mar 24, 2020 at 2:13 PM Philippe Mathieu-Daudé
<address@hidden> wrote:
>
> Hi Basil,
>
> On 3/11/20 6:04 PM, Basil Salman wrote:
> > Installation might fail if we don't wait for the provider
> > unregisteration process to finish.
> >
> > Signed-off-by: Sameeh Jubran <address@hidden>
> > Signed-off-by: Basil Salman <address@hidden>
> > ---
> > qga/installer/qemu-ga.wxs | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/qga/installer/qemu-ga.wxs b/qga/installer/qemu-ga.wxs
> > index 64bf90bd85..f6781752e6 100644
> > --- a/qga/installer/qemu-ga.wxs
> > +++ b/qga/installer/qemu-ga.wxs
> > @@ -81,7 +81,7 @@
> > Arguments="-d --retry-path"
> > >
> > </ServiceInstall>
> > - <ServiceControl Id="StartService" Start="install" Stop="both"
> > Remove="uninstall" Name="QEMU-GA" Wait="no" />
> > + <ServiceControl Id="StartService" Start="install" Stop="both"
> > Remove="uninstall" Name="QEMU-GA" Wait="yes" />
>
> As 'yes' is the default, can you simply remove 'Wait="no"'?
It looks like wixl doesn't follow the WiX default value though.
tools/wixl/msi.vala:
574 (Wait != null && !rec.set_int (5, Wait ? 1 : 0)) ||
>
> > </Component>
> > <?ifdef var.InstallVss?>
> > <Component Id="qga_vss_dll"
> > Guid="{CB19C453-FABB-4BB1-ABAB-6B74F687BFBB}">
> >
>
>
--
Marc-André Lureau