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

Fabiano Fidêncio fidencio at redhat.com
Wed Apr 27 12:21:15 UTC 2016


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) ...

>
> Regards,
> Daniel
> --
> |: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
> |: http://libvirt.org              -o-             http://virt-manager.org :|
> |: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
> |: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the virt-tools-list mailing list