<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/120579/">https://git.reviewboard.kde.org/r/120579/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 14th, 2014, 1:50 p.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, it's not installed as the WaylandServer still needs work and is not in a state yet to provide ABI.</p></pre>
</blockquote>
<p>On October 14th, 2014, 2:03 p.m. CEST, <b>Sebastian Kügler</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;">Well, neither is WaylandServer, the point at this point in time is to allow sharing code, no?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Is there another way to easily start a wayland server to I can run my autotests automatically?</p></pre>
</blockquote>
<p>On October 14th, 2014, 2:13 p.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;">that's why WaylandServer is not installed. The WaylandClient library is supposed to be ABI stable.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I want to get it into a state that WaylandServer can be ABI stable. But I cannot guarantee that I will be happy with the state for 5.1. It pretty much depends on how fast we get the interfaces implemented. At the moment lots is still stubs which makes it difficult to say "yeah, that API won't change anymore".</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For your current need I'd suggest to keep the patch locally to get it working and make proper CMake magic in kscreen. AFAIU you only need the server for the unit test, so the component should only be searched for tests and the respective tests should just not be built if the Component is not found. That way you should be able to write all the tests and they will just work (TM) once this component becomes public.</p></pre>
</blockquote>
<p>On October 14th, 2014, 3:03 p.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;">As you submitted part of the change I read that as the suggestion works for you?</p></pre>
</blockquote>
<p>On October 14th, 2014, 3:27 p.m. CEST, <b>Sebastian Kügler</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;">Ah, sorry, should have replied in-text (not just in-commit and assume you read my mind to fill in the blanks).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Well, the export header fix needed to go in, anyway, so that's pushed and out of the way.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">As to the other part, I suppose it's OK (I simply trust you there), it requires a little more attention to be paid on my side, but as the wayland backend in libkscreen is also still incumbent (and not quite near useful), I can live with that for now. That's assuming I can check for KF5::WaylandClient and KF5::WaylandServer separately in the cmake foo, I haven't checked that yet, the current check builds the wayland backend and the tests conditional if(Wayland_Client_FOUND AND KF5Wayland_FOUND), so there's no distinction between WaylandServer and WaylandClient, yet. As I said, haven't checked that yet.</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">unless there's a bug in the CMake foo in KWayland (I wouldn't be suprised if there were ;-) ), it should be possible to find separately. If not please report and we can fix it.</p></pre>
<br />
<p>- Martin</p>
<br />
<p>On October 14th, 2014, 2:49 p.m. CEST, Sebastian Kügler 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 kwin, Plasma and Martin Gräßlin.</div>
<div>By Sebastian Kügler.</div>
<p style="color: grey;"><i>Updated Oct. 14, 2014, 2:49 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kwayland
</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;">Install headers for WaylandServer</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'm not sure why this was commented (it didn't work, due to wrong
paths), but maybe there's another reason.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Anyway, I'd like to use these things for my unit tests in libkscreen, so
I don't have to start a Wayland server manually, and this seems to be
needed.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">In detail:
- actually install headers
- generate the export header into Wayland/Server
- include it from there</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;">Used the library from libkscreen, no problems so far (headers found, linker succeeds).</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>src/server/CMakeLists.txt <span style="color: grey">(066d564507633b42455482b441912359d7e65c74)</span></li>
<li>src/server/buffer_interface.h <span style="color: grey">(9cf84c5f3b62df6d53c36d4671302388a9adf261)</span></li>
<li>src/server/compositor_interface.h <span style="color: grey">(60ea01a59cea595df39d5ab74215f2de98fb390d)</span></li>
<li>src/server/display.h <span style="color: grey">(31cb24fc46576e94e72e637a81b99b8155fcbe9b)</span></li>
<li>src/server/output_interface.h <span style="color: grey">(eba9a32e5110a6892931665ff568e4322cd1c8e1)</span></li>
<li>src/server/seat_interface.h <span style="color: grey">(18d2dba38564df0519cd6f636f2dc527473c0b97)</span></li>
<li>src/server/shell_interface.h <span style="color: grey">(31f84044c7dcfb172e6a8a7134f9474ff973c5fd)</span></li>
<li>src/server/surface_interface.h <span style="color: grey">(31e0e75e3db534720900cf4458ea1c265f5570a7)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/120579/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>