<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/118846/">https://git.reviewboard.kde.org/r/118846/</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 20th, 2014, 7:05 p.m. CEST, <b>David Faure</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;">Strange, why wasn't this necessary in kdelibs4?</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;">see the linked bug report. It's basically a regression due to introducing platform detection (X11 vs. Wayland) in both klauncher and KWindowSystem. My first idea to fix it was to check for DISPLAY variable set instead of checking the platformName, but going deeper showed that it would fail in all the KWindowSystem checks, too.</pre>

<br />

<p>- Martin</p>

<br />

<p>On June 20th, 2014, 1:51 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 KDE Frameworks, David Faure and Harald Sitter.</div>

<div>By Martin Gräßlin.</div>

<p style="color: grey;"><i>Updated June 20, 2014, 1:51 p.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>

 <a href="https://bugs.kde.org/show_bug.cgi?id=336488">336488</a>

</div>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

kinit

</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;">Turn KLauncher into a QGuiApplication

klauncher uses the X11 windowing system which means it should be a

QGuiApplication. This is important as otherwise all the runtime checks

for whether X11 is used will fail. This is not only inside KLauncher

itself, but also in KWindowSystem.

BUG: 336488</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/klauncher/klauncher_main.cpp <span style="color: grey">(02e64b704308b6e815019e719eb7a37f3683127a)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/118846/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>