diff mbox

qemu-ga: Fixed minor version switch issue

Message ID 1452503561-29521-1-git-send-email-leonid@daynix.com
State New
Headers show

Commit Message

Leonid Bloch Jan. 11, 2016, 9:12 a.m. UTC
With automatically generated GUID, on minor version changes, an error
occurred, stating that there is a problem with the installer.
Now, a notification is shown, warning the user that another version of
this product is already installed, and that configuration or removal of
the existing version is possible through Add/Remove Programs on the
Control Panel (expected behavior).

Signed-off-by: Leonid Bloch <leonid@daynix.com>
---
 qga/installer/qemu-ga.wxs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Leonid Bloch Jan. 31, 2016, 2:18 p.m. UTC | #1
ping

http://patchwork.ozlabs.org/patch/565712

On Mon, Jan 11, 2016 at 11:12 AM, Leonid Bloch <leonid@daynix.com> wrote:

> With automatically generated GUID, on minor version changes, an error
> occurred, stating that there is a problem with the installer.
> Now, a notification is shown, warning the user that another version of
> this product is already installed, and that configuration or removal of
> the existing version is possible through Add/Remove Programs on the
> Control Panel (expected behavior).
>
> Signed-off-by: Leonid Bloch <leonid@daynix.com>
> ---
>  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 9473875..7f92891 100644
> --- a/qga/installer/qemu-ga.wxs
> +++ b/qga/installer/qemu-ga.wxs
> @@ -41,7 +41,7 @@
>
>    <Product
>      Name="QEMU guest agent"
> -    Id="*"
> +    Id="{DF9974AD-E41A-4304-81AD-69AA8F299766}"
>      UpgradeCode="{EB6B8302-C06E-4BEC-ADAC-932C68A3A98D}"
>      Manufacturer="$(env.QEMU_GA_MANUFACTURER)"
>      Version="$(env.QEMU_GA_VERSION)"
> --
> 2.4.3
>
>
Michael Roth Feb. 10, 2016, 5:43 p.m. UTC | #2
Quoting Leonid Bloch (2016-01-11 03:12:41)
> With automatically generated GUID, on minor version changes, an error
> occurred, stating that there is a problem with the installer.
> Now, a notification is shown, warning the user that another version of
> this product is already installed, and that configuration or removal of
> the existing version is possible through Add/Remove Programs on the
> Control Panel (expected behavior).
> 
> Signed-off-by: Leonid Bloch <leonid@daynix.com>

Reviewed-by: Michael Roth <mdroth@linux.vnet.ibm.com>

> ---
>  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 9473875..7f92891 100644
> --- a/qga/installer/qemu-ga.wxs
> +++ b/qga/installer/qemu-ga.wxs
> @@ -41,7 +41,7 @@
> 
>    <Product
>      Name="QEMU guest agent"
> -    Id="*"
> +    Id="{DF9974AD-E41A-4304-81AD-69AA8F299766}"
>      UpgradeCode="{EB6B8302-C06E-4BEC-ADAC-932C68A3A98D}"
>      Manufacturer="$(env.QEMU_GA_MANUFACTURER)"
>      Version="$(env.QEMU_GA_VERSION)"
> -- 
> 2.4.3
>
Michael Roth Feb. 25, 2016, 2:07 a.m. UTC | #3
Quoting Leonid Bloch (2016-01-11 03:12:41)
> With automatically generated GUID, on minor version changes, an error
> occurred, stating that there is a problem with the installer.
> Now, a notification is shown, warning the user that another version of
> this product is already installed, and that configuration or removal of
> the existing version is possible through Add/Remove Programs on the
> Control Panel (expected behavior).
> 
> Signed-off-by: Leonid Bloch <leonid@daynix.com>

Thanks, applied to qga tree:

  https://github.com/mdroth/qemu/commits/qga

> ---
>  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 9473875..7f92891 100644
> --- a/qga/installer/qemu-ga.wxs
> +++ b/qga/installer/qemu-ga.wxs
> @@ -41,7 +41,7 @@
> 
>    <Product
>      Name="QEMU guest agent"
> -    Id="*"
> +    Id="{DF9974AD-E41A-4304-81AD-69AA8F299766}"
>      UpgradeCode="{EB6B8302-C06E-4BEC-ADAC-932C68A3A98D}"
>      Manufacturer="$(env.QEMU_GA_MANUFACTURER)"
>      Version="$(env.QEMU_GA_VERSION)"
> -- 
> 2.4.3
>
Leonid Bloch Feb. 25, 2016, 8:15 a.m. UTC | #4
On Feb 25, 2016 4:09 AM, "Michael Roth" <mdroth@linux.vnet.ibm.com> wrote:
>
> Quoting Leonid Bloch (2016-01-11 03:12:41)
> > With automatically generated GUID, on minor version changes, an error
> > occurred, stating that there is a problem with the installer.
> > Now, a notification is shown, warning the user that another version of
> > this product is already installed, and that configuration or removal of
> > the existing version is possible through Add/Remove Programs on the
> > Control Panel (expected behavior).
> >
> > Signed-off-by: Leonid Bloch <leonid@daynix.com>
>
> Thanks, applied to qga tree:
>
>   https://github.com/mdroth/qemu/commits/qga

Thanks, Michael.

>
> > ---
> >  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 9473875..7f92891 100644
> > --- a/qga/installer/qemu-ga.wxs
> > +++ b/qga/installer/qemu-ga.wxs
> > @@ -41,7 +41,7 @@
> >
> >    <Product
> >      Name="QEMU guest agent"
> > -    Id="*"
> > +    Id="{DF9974AD-E41A-4304-81AD-69AA8F299766}"
> >      UpgradeCode="{EB6B8302-C06E-4BEC-ADAC-932C68A3A98D}"
> >      Manufacturer="$(env.QEMU_GA_MANUFACTURER)"
> >      Version="$(env.QEMU_GA_VERSION)"
> > --
> > 2.4.3
> >
>
diff mbox

Patch

diff --git a/qga/installer/qemu-ga.wxs b/qga/installer/qemu-ga.wxs
index 9473875..7f92891 100644
--- a/qga/installer/qemu-ga.wxs
+++ b/qga/installer/qemu-ga.wxs
@@ -41,7 +41,7 @@ 
 
   <Product
     Name="QEMU guest agent"
-    Id="*"
+    Id="{DF9974AD-E41A-4304-81AD-69AA8F299766}"
     UpgradeCode="{EB6B8302-C06E-4BEC-ADAC-932C68A3A98D}"
     Manufacturer="$(env.QEMU_GA_MANUFACTURER)"
     Version="$(env.QEMU_GA_VERSION)"