Re: [PATCH qxl] Xspice: Don't set defaults for the options.

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

 



On Mon, Aug 31, 2015 at 03:18:50PM +0200, Francois Gouget wrote:
> On Mon, 10 Aug 2015, Christophe Fergeau wrote:
> 
> > On Mon, Jun 08, 2015 at 05:52:59PM +0200, Francois Gouget wrote:
> > > Otherwise they override Spice server's real builtin defaults, the Xorg configuration file settings, and even the XSPICE_XXX environment variables.
> > > ---
> > > 
> > > Without this patch, calling Xspice _without_ the '--streaming-video' 
> > > option forces this setting to 'filter', overriding the 
> > > XSPICE_STREAMING_VIDEO environment variable and the SpiceStreamingVideo 
> > > spiceqxl.xorg.conf setting.
> > 
> > fwiw this belongs to the commit log, not below '---'
> 
> It mostly says the same thing as in the commit message but in a more 
> concrete way. I've merged the two together.
> 
> 
> > >  parser.add_argument('--image-compression',
> > >                      choices = ['off', 'auto_glz', 'auto_lz', 'quic',
> > >                                 'glz', 'lz'],
> > > -                    default='auto_glz', help='auto_glz by default')
> > > +                    help='auto_glz by default')
> > 
> > I think these help strings should be reworked now, like 'use
> > spice-server defaults (auto_glz)' or something like that.
> 
> That feels like a separate issue so it should go in a separate patch.

This particular commit goes from 'hardcoded defaults' to 'spice-server
defaults' so I don't see it as that separate, but I'm  fine with
keepinng that for later.

Christophe

Attachment: pgpQrOx051vRe.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]