<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>
</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;">As you submitted part of the change I read that as the suggestion works for you?</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>