Re: Xwayland fixes wrt eglstream support

Submitted by Olivier Fourdan on June 5, 2018, 5:37 p.m.

Details

Reviewer None
Submitted June 5, 2018, 5:37 p.m.
Last Updated June 5, 2018, 5:38 p.m.
Revision 1
Root msg-id(s): 20180605173758.31607-1-ofourdan@redhat.com

Cover Letter(s)

Revision 1 
      Hi,

This is a follow-up on https://patchwork.freedesktop.org/series/43704/
after Lyude and Emil reviews.

Emil, for 1/5, I changed the error message if eglstream wasn't enabled at
build time to:

  "xwayland glamor: this build does not have eglstream support"

which sounds to me like the most accurate description :)

I rephrased the commit messages in 1/3 and in 5/5 as per your wording.

I also fixed a few typos in the commit messages and added the R-b from
both Lyude and yourself.

Cheers,
Olivier

Olivier Fourdan (5):
  xwayland: allow "-eglstream" option
  xwayland: "EGL_EXT_device_base" required for EGLStream
  xwayland: process Wayland events after adding screen
  xwayland: do not disable glamor if EGLStream failed
  xwayland: make xwl_output_get_xdg_output() static

 hw/xwayland/xwayland-glamor.c |  3 +++
 hw/xwayland/xwayland-output.c |  4 +++-
 hw/xwayland/xwayland.c        | 24 ++++++++++++------------
 hw/xwayland/xwayland.h        |  1 -
 4 files changed, 18 insertions(+), 14 deletions(-)

Revisions