[virt-tools-list] [PATCH 2/3] Remove nsis installer

[Christophe Fergeau]

On Wed, Apr 27, 2016 at 02:21:15PM +0200, Fabiano Fidêncio wrote:
> On Tue, Apr 26, 2016 at 5:03 PM, Daniel P. Berrange <berrange at redhat.com> wrote:
> > On Tue, Apr 26, 2016 at 04:43:53PM +0200, Fabiano Fidêncio wrote:
> >> On Tue, Apr 26, 2016 at 4:34 PM, Daniel P. Berrange <berrange at redhat.com> wrote:
> >> > On Tue, Apr 26, 2016 at 04:29:07PM +0200, Christophe Fergeau wrote:
> >> >> On Tue, Apr 26, 2016 at 04:16:41PM +0200, Fabiano Fidêncio wrote:
> >> >> > The only reason for us to keep maintaining the nsis installer was the
> >> >> > activex plugin (spicex), which requires those nsis based installers.
> >> >> > As the next release of RHEV/oVirt won't use the activex plugin (spicex)
> >> >> > let's completely remove the nsis installer from our tree and focus on
> >> >> > only maintain the msi installer.
> >> >>
> >> >> Given that oVirt 4.0 is not released yet, might make sense to hold that
> >> >> for a bit? Maybe some last minute issue will prevent that removal.
> >> >
> >> > Does anyone know what the newest version of virt-viewer is that oVirt
> >> > has been shipping ?  It could be valid to say that if they want NSIS
> >> > they should stick on the v2.0-maint of v3.0-maint branches instead
> >> > of updating to the virt-viewer 4.0 release .
> >>
> >> 2.0 with a bunch of patch on top, considering the client is the same
> >> one used for RHEVM.
> >
> > IMHO, since they've not even upgraded to 3.0 release version, I think
> > we're fine to drop nsis in the virt-viewer-4.0 version, without waiting
> > for the next oVirt release.
> 
> Christophe? I'd go for the patch as well (but you know, the patch is mine) ...

Oh sorry for not following up, I'm fine with this plan too.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20160427/216239f5/attachment.sig>