<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/117781/">https://git.reviewboard.kde.org/r/117781/</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 75ba2ae6c1f35056cbb03de008c3605e10b9990e by Luigi Toscano to branch master.</pre>
 <br />









<p>- Commit Hook</p>


<br />
<p>On April 26th, 2014, 1:08 a.m. UTC, Luigi Toscano 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 Documentation and KDE Frameworks.</div>
<div>By Luigi Toscano.</div>


<p style="color: grey;"><i>Updated April 26, 2014, 1:08 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kdoctools
</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;">Install again common/, but:
- under a different prefix
- change the structure to allow the installation of the localized common files for other languages. Those files currently live in SVN under trunk/l10n-kde4/<lang>/docs/common but those directories will be removed for kf5 and the content, if any, will be imported here.

This is the combination of four commits which can be checked here:
http://quickgit.kde.org/?p=clones%2Fkdoctools%2Fltoscano%2Fkdoctools-tosky.git&a=log&h=9cd70da7d44147eeb0b119f77bb08e8bb00e87d7

Commit logs (older first):
---
Move the definition of language list to the parent CMakeLists.txt

The list can be then reused (namely by common/)
---    
Move "en" common doc files to own directory
---    
Install common doc files for all the languages

common/CMakeLists.txt has been completely rewritten; it now supports
a set of language directories and all the files included there.
Right now only en/ is defined, other common files will be added
using the corresponding files from trunk/l10n-kde4/<lang>/docs/common/
(SVN).
---    
Enable common/ for a different prefix, adapt kde-chunk.xsl

Common files are installed again as it was in kdelibs4, but under
a different prefix (kdoctools5-common) to avoid conflicts with the
"old" common directory from kdelibs4.
kde-chunk.xsl has been adapted to use this new prefix.



</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;">- The files are installed under the common doc prefix
- `kioclient cat help:/konsole/credits.html` shows the usage of the new prefix
- `kioclient cat help:/kdoctools5-common/gpl-license.html` works</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>common/artistic-license.html <span style="color: grey">(2069b4e)</span></li>

 <li>CMakeLists.txt <span style="color: grey">(3e41a26)</span></li>

 <li>common/CMakeLists.txt <span style="color: grey">(4228625)</span></li>

 <li>common/block_title_bottom.png <span style="color: grey">(c0c0af5)</span></li>

 <li>common/block_title_mid.png <span style="color: grey">(f1ce933)</span></li>

 <li>common/block_title_top.png <span style="color: grey">(2344376)</span></li>

 <li>common/bsd-license.html <span style="color: grey">(422bb33)</span></li>

 <li>common/en/artistic-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/block_title_bottom.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/block_title_mid.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/block_title_top.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/bsd-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/fdl-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/fdl-notice.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/gpl-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/kde-default.css <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/kde-docs.css <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/kde_logo.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/kde_logo_bg.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/kmenu.png <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/lgpl-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/qpl-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/top-kde.jpg <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/top-left.jpg <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/top-right.jpg <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/top.jpg <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/x11-license.html <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/en/xml.dcl <span style="color: grey">(PRE-CREATION)</span></li>

 <li>common/fdl-license.html <span style="color: grey">(9ce378a)</span></li>

 <li>common/fdl-notice.html <span style="color: grey">(862f09e)</span></li>

 <li>common/gpl-license.html <span style="color: grey">(26ae3af)</span></li>

 <li>common/kde-default.css <span style="color: grey">(24ef928)</span></li>

 <li>common/kde-docs.css <span style="color: grey">(12291a4)</span></li>

 <li>common/kde_logo.png <span style="color: grey">(336bfd8)</span></li>

 <li>common/kde_logo_bg.png <span style="color: grey">(6cf47f1)</span></li>

 <li>common/kmenu.png <span style="color: grey">(ff077c2)</span></li>

 <li>common/lgpl-license.html <span style="color: grey">(343576d)</span></li>

 <li>common/qpl-license.html <span style="color: grey">(19a863b)</span></li>

 <li>common/top-kde.jpg <span style="color: grey">(48967d3)</span></li>

 <li>common/top-left.jpg <span style="color: grey">(23c31c7)</span></li>

 <li>common/top-right.jpg <span style="color: grey">(22d2a29)</span></li>

 <li>common/top.jpg <span style="color: grey">(8be8caa)</span></li>

 <li>common/x11-license.html <span style="color: grey">(444a27b)</span></li>

 <li>common/xml.dcl <span style="color: grey">(fed2103)</span></li>

 <li>src/CMakeLists.txt <span style="color: grey">(a2bb02b)</span></li>

 <li>src/customization/kde-chunk.xsl <span style="color: grey">(eda3d38)</span></li>

</ul>

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







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








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