<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/118028/">https://git.reviewboard.kde.org/r/118028/</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;">This review has been submitted with commit 1f4d1240b3ae63457492d2288a993bbfc4760a8e by Friedrich W. H. Kossebau to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On May 6th, 2014, 11:04 p.m. UTC, Friedrich W. H. Kossebau 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 Calligra and Sven Langkamp.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated May 6, 2014, 11:04 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</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;">It was found (by missing deps in OpenSUSE packages) that Okular crashes (for some yet to be investigated reason) if the Calligra Okular ODP plugin is installed, but not the Stage part.
This patch notes this runtime dependency in the README.PACKAGERS.
It also splits the definition of the PPT and PPTX support into separate desktop files, only installed if the needed filters are also built.
Possibly should be also backported to 2.8</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">(a32d6b4)</span></li>
<li>README.PACKAGERS <span style="color: grey">(2a686d6)</span></li>
<li>cmake/productsets/all.cmake <span style="color: grey">(8e82b40)</span></li>
<li>cmake/productsets/desktop.cmake <span style="color: grey">(4e85bc8)</span></li>
<li>extras/okularodpgenerator/libokularGenerator_odp.desktop <span style="color: grey">(964bf21)</span></li>
<li>extras/okularodpgenerator/okularApplication_odp.desktop <span style="color: grey">(788825e)</span></li>
<li>extras/okularodpgenerator/okularOdp.desktop <span style="color: grey">(7b1a8d3)</span></li>
<li>filters/stage/powerpoint/CMakeLists.txt <span style="color: grey">(8d7df54)</span></li>
<li>filters/stage/powerpoint/libokularGenerator_powerpoint_calligra.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>filters/stage/powerpoint/okularApplication_powerpoint_calligra.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>filters/stage/powerpoint/okularPowerpoint_calligra.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>filters/stage/pptx/CMakeLists.txt <span style="color: grey">(9a07c37)</span></li>
<li>filters/stage/pptx/libokularGenerator_pptx_calligra.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>filters/stage/pptx/okularApplication_pptx_calligra.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>filters/stage/pptx/okularPptx_calligra.desktop <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118028/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>