<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/118694/">https://git.reviewboard.kde.org/r/118694/</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 12th, 2014, 1:36 p.m. CEST, <b>David Edmundson</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;">Ship it from me, but I'd like you to get approval from one of the Windows people first.</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;">I think we have that already: https://mail.kde.org/pipermail/plasma-devel/2014-June/032269.html</pre>
<br />
<p>- Martin</p>
<br />
<p>On June 12th, 2014, 1:32 p.m. CEST, Martin Gräßlin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated June 12, 2014, 1:32 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-desktop
</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;">X11/XCB becomes a required component
So far plasma-desktop supported to be built without X11 support (Win32),
but that was completely broken (X11Extras was required). Based on
discussion in [1] X11/XCB becomes a required component.
This means that we should not have any components in the build system
which are built optionaly depending on whether X11 is available. Ifdefs
in the code should stay, though.
For the future we will turn X11 in a optional dependency again. But it
will never be that a component won't be built if we are building without
X11. Instead there will be a required Wayland backend and this means all
X11 specific code needs to become platform aware.
So the way forward is:
1. X11 required
2. X11 required, Wayland optional
3. X11 required, Wayland required
4. Wayland required, X11 optional
[1] https://mail.kde.org/pipermail/plasma-devel/2014-June/032259.html</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">(399de771e17b265c1e5a223ad128ae2d17c00d22)</span></li>
<li>applets/CMakeLists.txt <span style="color: grey">(a5f35f77aead140054696ca4b3122fb177c70f1d)</span></li>
<li>kcms/CMakeLists.txt <span style="color: grey">(8024367f52b4fd3f757d4e5c5c851a204f4fd74b)</span></li>
<li>kcms/componentchooser/CMakeLists.txt <span style="color: grey">(66f0fece8b786cf277cf41e6bef0cf03b3f619d1)</span></li>
<li>kcms/componentchooser/componentservices/CMakeLists.txt <span style="color: grey">(15cca45a12e2e61c721f092b26fa5953701ee958)</span></li>
<li>kcms/style/CMakeLists.txt <span style="color: grey">(423241b8127491853d8afa75bb8b5d78c9009dca)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118694/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>