[PATCH xserver 0/2] xwayland: couple of EGL backend API cleanup

[Olivier Fourdan]

Hi,

On Fri, Jun 15, 2018 at 9:46 AM, Olivier Fourdan <ofourdan at redhat.com> wrote:
> Hi Emil,
>
> On Thu, Jun 14, 2018 at 7:38 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> What's the blocker of pushing these - short on commit access, other?
>> If the former, that should be addressed. You have done some serious
>> work on Xwayland.
>
> I don't think I have commit access myself, never tried actually, never dare to!

Well, I could think of a reason why it hasn't been picked up yet...

Those patch series are quite a lot of refactoring, and considering we
haven't branched 1.20 yet, it might be unclear if this is material for
master or xserver-1.20-branch (once it's created).

FWIW, I reckon those series are a feature fix primarily, i.e. enable
EGLStream support automatically when needed without requiring to
change all compositors to add “-eglstream” explicitly, which would
crash in different and creative ways with the current code (unknown
option if Xwayland was built without EGLStream support, or crash if
EGLStream support is there in Xwayland but the hardware isn't).

So, IMHO, these patches should be part of 1.20 at some point, but I
understand that it's quite a lot of change for a stable tree (even if
released only recently) so I would leave that to whoever takes the
role of release manager to decide :)

Cheers,
Olivier