Re: [PATCHv2 virt-viewer 0/4] Add version field in connection file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Mensaje original -----
> On Fri, Mar 08, 2013 at 03:15:33PM +0100, Marc-André Lureau wrote:
> > Hi,
> > 
> > It might be worthwhile to check for required client version before
> > establishing connection.
> > 
> > (I don't think it's worth to check for each capabilities and
> > protocol
> >  version, this would only explose the number of combinations. I
> >  think
> >  it is better to assume that the min. requirement are checked
> >  during
> >  configure - assuming the respective backend needed are
> >  compiled-in,
> >  otherwise there is already a protocol required check error)
> 
> As with previous posting, IMHO, feature based checks are far better
> than any version checks, precisely because of the assumption you
> mention above - we should not be assuming people turn on every
> feature
> at configure time.

Then again, I am in favour of removing configure options.

And feature checks are not user friendly. We want a simple dialog saying "You need a newer version of virt-viewer". Not a dialog saying something like "You need spice-gtk with dynamic multi-monitor support"...

This is really targeting Windows use case, for which we will provide an "official" build.

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list



[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux