On Tue, Jul 17, 2018 at 09:51:28AM -0400, Frediano Ziglio wrote:
> >
> > spice-html5 does not support multiple display channels, and will only
> > use the first one offered by spice-server, and ignore the others. This
> > can cause issues when using a VM with a configured but unused
> > org.spice-space.stream.0 spice port as this will create a 'fake' display
> > channel which is offered by spice-server before the usual qxl display
> > channel.
> > This commit favours the display channel with id 0 rather than picking
> > the first one.
> >
> > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> > ---
> > main.js | 7 ++++---
> > 1 file changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/main.js b/main.js
> > index 173ff97..5ddc896 100644
> > --- a/main.js
> > +++ b/main.js
> > @@ -146,10 +146,11 @@ SpiceMainConn.prototype.process_channel_message =
> > function(msg)
> > };
> > if (chans.channels[i].type == SPICE_CHANNEL_DISPLAY)
> > {
> > - if (this.display !== undefined)
> > - this.log_warn("The spice-html5 client does not handle
> > multiple heads.");
> > - else
> > + if (chan_id == 0) {
>
> Did you try it? I think is conn.chan_id or chans.channels[i].id
I did try some variant of it, but clearly not this one as chan_id is
indeed invalid. v2 sent, thanks!
Christophe
>
> > this.display = new SpiceDisplayConn(conn);
> > + } else {
> > + this.log_warn("The spice-html5 client does not handle
> > multiple heads.");
> > + }
> > }
> > else if (chans.channels[i].type == SPICE_CHANNEL_INPUTS)
> > {
>
> Frediano
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel