<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="http://git.reviewboard.kde.org/r/106970/">http://git.reviewboard.kde.org/r/106970/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Over a week and still no comments. If no one objects in the next few days, I'm simply gonna push this without any "Ship It!".</pre>
 <br />







<p>- Jon</p>


<br />
<p>On October 20th, 2012, 6:56 p.m., Jon Severinsson wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KDE Frameworks and Phonon.</div>
<div>By Jon Severinsson.</div>


<p style="color: grey;"><i>Updated Oct. 20, 2012, 6:56 p.m.</i></p>






<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;">This patch is based on my "Phonon five merge and cleanup" patch, and is a step towards building Phonon five on Qt5.

On Qt4 the build system will now insist on "libinqt5" being installed,
which is located in the kdelibs repository in the frameworks branch.
There is, however, no dependancy on the actual KDE Frameworks 5, and
libinqt5 can be build and installed standalone.</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>cmake/FindPhononInternal.cmake <span style="color: grey">(20220a3)</span></li>

 <li>demos/metadatareader/CMakeLists.txt <span style="color: grey">(fe6eb60)</span></li>

 <li>demos/phota/CMakeLists.txt <span style="color: grey">(2306c7e)</span></li>

 <li>demos/simplecapture/CMakeLists.txt <span style="color: grey">(4a2e1b0)</span></li>

 <li>demos/simpleplayer/CMakeLists.txt <span style="color: grey">(6824142)</span></li>

</ul>

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




  </td>
 </tr>
</table>








  </div>
 </body>
</html>