D17122: Add option to use wl_display_add_socket_auto

Fabian Vogt noreply at phabricator.kde.org
Fri Jan 25 12:15:47 GMT 2019


fvogt added inline comments.

INLINE COMMENTS

> romangg wrote in test_display.cpp:223
> This fails when one tries to run the test inside another Wayland session, which already uses the wayland-0 socket name.
> 
> What you could do here is loop until you find the first non-existing `wayland-x` in the runtime dir. Then do it one more time from there until you find the second `wayland-y`. Remember x,y and compare these below against the created socket names.

That won't work reliably either though - if wayland-0 is free, but wayland-1 is used, it would pick wayland-0 and wayland-2. Maybe it should just check that starting both displays at the same time succeeds and that their socket names are not equal.

REPOSITORY
  R127 KWayland

REVISION DETAIL
  https://phabricator.kde.org/D17122

To: fvogt, #kwin, #plasma, romangg
Cc: zzag, romangg, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190125/d9419761/attachment.html>


More information about the Kde-frameworks-devel mailing list