Wayland Compositor and X11 Window Manager
Let's get it in 5.17 beta branch then as well. Tested it on X11 and Wayland and no related compile warnings.
I have to admit in hindsight that I was maybe wrong on this one not being in 5.17. Since only having the change on master but not in the 5.17 branch could complicate porting back bug fixes. Would you guys agree to that? And since we are still at the beginning of beta phase shall we apply it also to 5.17 branch?
At this point, no.
@davidedmundson Is this 5.17 stuff?
Nitpick: please use imperative mood in the subject line.
Do you have commit access?
I changed my version to version of zzag, because his code is more simple. I have tested both versions.
env var should be called KWIN_DRM_DEVICE_NODE
I'm ok with having this as a short-term fix but long-term this env var should go away again and be replaced with a better selection mechanism. So no guarantee that it will be available after 5.18 LTS.
Please more verbose summary.
What about simplifying it a bit as well? e.g.
Tue, Sep 17
Please next time besides a title add a meaningful description if you push a patch with more than a trivial change and without review. I assume it was just a mistake that you also didn't link back to the review in the commit bccbb8f3a5f8.
Mon, Sep 16
I still think this is a great idea.