Comment 119 for bug 36812

Revision history for this message
Bryce Harrington (bryce) wrote :

@Alexey Kozlov, thanks for your well written summary of the problem.

The one warning flag is the "contradicts spec" issue (which may be what has blocked this from being taken up 'til now), however the exact words from upstream about this seem less dire:

 The patch looks pretty much fine to me, except that I'd just hardcode
 release-only and don't try to make it configurable. I'm not entirely
 sure about the bit marked with KLUDGE, and would like to find a better
 way to do it, but if that's fixed (I'll try to have a look this weekend
 or next, depending on time), my only objection to merging it is that it
 explicitly contradicts the spec. I'm not entirely convinced that's a
 dealbreaker though.

I can understand that voiding the spec is a blocker for upstream, but at the distro level there's a little bit more flexibility, especially given the top-level direction that we'll eventually move away from X11 towards something Wayland-ish anyway.

It sounds like the next step required for this patch is to gain some more widespread testing. Since the change is behavioral, it seems low risk for causing crashes or other critical issues, so tentatively I'm willing to stick it in natty and see how it goes.