<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/128073/">https://git.reviewboard.kde.org/r/128073/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 1st, 2016, 10:13 a.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">-2, see also https://bugs.kde.org/show_bug.cgi?id=361954 for explanation why we don't want this.</p></pre>
</blockquote>
<p>On June 1st, 2016, 10:23 a.m. CEST, <b>Pino Toscano</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Nowhere there is discussed (only briefly mentioned) about support with non-Linux platforms; considering libkscreen is a library used by components (and also external ones, like the display configuration in LxQt), then it makes sense to me making it portable.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Making wayland mandatory in libkscreen (and also in other modules, like plasma-integration, kwin, kscreenlocker) means killing outright any attempt to make Plasma build on non-Linux platforms. Please reconsider this, thanks.</p></pre>
</blockquote>
<p>On June 1st, 2016, 10:29 a.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">which non-Linux platforms which we care about don't have Wayland? Before making KWayland a mandatory dependency in Plasma we did check with distro packagers. Everyone was fine with it.</p></pre>
</blockquote>
<p>On June 1st, 2016, 10:44 a.m. CEST, <b>Pino Toscano</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">At least to my reading of Wayland sources, to build the libraries you need features like signalfd and timerfd, which are non-POSIX and available only on Linux.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Can you please point me to the discussion you had with distro packagers?</p></pre>
</blockquote>
<p>On June 1st, 2016, 10:56 a.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">There are non-linux which have successfully run Weston on a DRM backend. That was in the news a few weeks back, see e.g. http://lists.dragonflybsd.org/pipermail/users/2016-May/249620.html</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So let's go from "theoretically not possible" to "practically not possible". Where is it not possible to have a Wayland dependency?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Concerning the discussion: a quick search showed me https://mail.kde.org/pipermail/kde-distro-packagers/2015-July/000076.html, but there must be an older one already as we use it in plasma-workspace (kscreenlocker) since Plasma 5.2 IIRC. This makes it difficult for me to point you to the discussion as it's really a long time ago.</p></pre>
</blockquote>
<p>On June 1st, 2016, 10:57 a.m. CEST, <b>Bhushan Shah</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">https://mail.kde.org/pipermail/release-team/2015-July/008725.html</p></pre>
</blockquote>
<p>On June 1st, 2016, 11:08 a.m. CEST, <b>Pino Toscano</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">https://mail.kde.org/pipermail/kde-distro-packagers/2015-July/000076.html
https://mail.kde.org/pipermail/release-team/2015-July/008725.html</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The only party involved there is a Slackware packager, which is a Linux distro (where the Wayland libraries can be built fine). I don't see anyone mentioning non-Linux support.</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">There are non-linux which have successfully run Weston on a DRM backend. That was in the news a few weeks back, see e.g. http://lists.dragonflybsd.org/pipermail/users/2016-May/249620.html</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">That's the second attempt at this I see (the first one was beginning of 2013 IIRC), yet none of this code as been upstreamed yet.</p></pre>
</blockquote>
<p>On June 1st, 2016, 11:43 a.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I don't see anyone mentioning non-Linux support.</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">All we can do is ask, sorry.</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">That's the second attempt at this I see (the first one was beginning of 2013 IIRC), yet none of this code as been upstreamed yet.</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Well I find it strange that I got code from BSD users for new KWin. If it were impossible to build KWin anymore that wouldn't work ;-)</p></pre>
</blockquote>
<p>On June 1st, 2016, 11:52 a.m. CEST, <b>Pino Toscano</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">All we can do is ask, sorry.</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Possibly not closing the door would help as well.</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Well I find it strange that I got code from BSD users for new KWin. If it were impossible to build KWin anymore that wouldn't work ;-)</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Well, I still see that kwin has kwayland and kscreenlocker as mandatory build dependencies, and kscreenlocker has kwayland as mandatory build dependency. Hence they are using non-upstreamed patches, which is something I don't want to rely on.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Do you have pointers to these contributions for BSD?</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Possibly not closing the door would help as well.</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">There's a time to discuss things. A year later it's sometimes too late. I can understand that this is a tricky situation, but it's also for us rather tricky if we would want to paddle back now. It also raises the question why we should ask at all. If I ask and don't get an answer, I have to interpret it as it won't cause problems.</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Well, I still see that kwin has kwayland and kscreenlocker as mandatory build dependencies,</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">And in KWin we cannot go back anymore. KWin is a Wayland compositor which happened to be an X11 window manager in it's past. We can talk about making X11 optional, but not Wayland.</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">and kscreenlocker has kwayland as mandatory build dependency.</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">KScreenLocker uses kwayland as a private and secure communication channel between deamon and greeter. Removing it would break kscreenlocker, reverting that part would result in security vulnerabilities. In addition it has Wayland specific code for KWin on Wayland.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Sorry also like in the case of KWin: it's too late to go back.</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Do you have pointers to these contributions for BSD?</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">e.g. https://phabricator.kde.org/D1425 - but there were more discussions resulting in me fixing stuff, e.g. https://git.reviewboard.kde.org/r/126182/</p></pre>
<br />
<p>- Martin</p>
<br />
<p>On June 1st, 2016, 10:10 a.m. CEST, Pino Toscano wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma, Solid, Daniel Vrátil, Martin Gräßlin, and Sebastian Kügler.</div>
<div>By Pino Toscano.</div>
<p style="color: grey;"><i>Updated June 1, 2016, 10:10 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
libkscreen
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Look for KF5Wayland as optional package, not building all the Wayland-related code (backend, additions to tests, DPMS support in kscreen-doctor) if it is not available.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This allows libkscreen to build again on platforms without Wayland, such as non-Linux or old Linux distros.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Builds fine.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(9c834d9500fa0b89764516464bd58e835671977b)</span></li>
<li>autotests/CMakeLists.txt <span style="color: grey">(2c8ee3d2021b1222072d6a1faafaff1b336d27e3)</span></li>
<li>backends/CMakeLists.txt <span style="color: grey">(3563e138454edcc0c874237745123b09077d067c)</span></li>
<li>src/doctor/CMakeLists.txt <span style="color: grey">(7294d88af127a6604bc7e48e16342c93f152de8c)</span></li>
<li>src/doctor/doctor.cpp <span style="color: grey">(a156f006212e63c155a6f643929d0a5880275731)</span></li>
<li>tests/CMakeLists.txt <span style="color: grey">(1489d21383e13cd1431a81d48b4a272c28302041)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128073/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>