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









<p>- Commit Hook</p>


<br />
<p>On February 27th, 2014, 10:40 p.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 Feb. 27, 2014, 10:40 p.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;">- rename the DTD file (the old one will be kept for for compatibility in
  kde4support)


- adapt the existing documentation: -> please note that these changes (usage of the new DTD) should be applied to all the documentation shipped with other Frameworks and with the documentation of modules that do not require KDE4Support (see next RR).<--
The change should be mostly a matter of replacing the DTD, as DocBookXML 4.5 is backward compatible with 4.2 according the specifications.


This RR comprises the steps 3a) and (limited to the current module) 3c) of the plan described here: 
http://mail.kde.org/pipermail/kde-frameworks-devel/2014-February/011978.html</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;">Compilation (including the other RR)</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/CMakeLists.txt <span style="color: grey">(0329bd3)</span></li>

 <li>docs/meinproc5/man-meinproc5.8.docbook <span style="color: grey">(77799b4)</span></li>

 <li>docs/qt5options/man-qt5options.7.docbook <span style="color: grey">(7afbf07)</span></li>

 <li>docs/kf5options/man-kf5options.7.docbook <span style="color: grey">(cb7973d)</span></li>

 <li>cmake/FindDocBookXML4.cmake <span style="color: grey">(eb4bfd8)</span></li>

 <li>docs/checkXML5/man-checkXML5.1.docbook <span style="color: grey">(68509b9)</span></li>

 <li>src/customization/catalog.xml <span style="color: grey">(229ae70)</span></li>

 <li>src/customization/dtd/kdedbx45.dtd.cmake <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/customization/dtd/kdex.dtd.cmake <span style="color: grey">(c2f7b2c)</span></li>

 <li>src/man-template.docbook <span style="color: grey">(bdc88c7)</span></li>

 <li>src/template.docbook <span style="color: grey">(08762e5)</span></li>

</ul>

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







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








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