<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/114979/">https://git.reviewboard.kde.org/r/114979/</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 43eec02513ab92b465e020777f10bf7457c01d64 by Alex Merry to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On January 15th, 2014, 2:09 p.m. UTC, Alex Merry 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 and Aurélien Gâteau.</div>
<div>By Alex Merry.</div>
<p style="color: grey;"><i>Updated Jan. 15, 2014, 2:09 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kapidox
</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;">Do not install licenses
Firstly, we should certainly not be installing something as generic as
standard license texts to a location as generic as share/LICENSES.
Secondly, it is not our job to install license text at all. We provide
licenses with all parts of our source distributions, and it is up to the
producers of binary distributions to decide on the best way to make
license text available for those.
Note that the combination of this and https://git.reviewboard.kde.org/r/114971/ will result in nothing being installed by kapidox at all. I eventually intend to install the apidox generation scripts themselves, along with the necessary data files.</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">(02114642f0713e44f581e58c4d47192b4db2d238)</span></li>
<li>licenses/ARTISTIC <span style="color: grey">(8f9bdefc4db9428a02812c0d24c887b0f3d7ef40)</span></li>
<li>licenses/BSD <span style="color: grey">(cca2a5c9abb92e2f4d4d2ad7085dcd9f59809bd0)</span></li>
<li>licenses/CMakeLists.txt <span style="color: grey">(ffc215b208cd7ebb5318e1283e306d24d396dc0d)</span></li>
<li>licenses/GPL_V2 <span style="color: grey">(64a675c0f0e75697595e31e7c3ed75dbbe65e567)</span></li>
<li>licenses/GPL_V3 <span style="color: grey">(94a9ed024d3859793618152ea559a168bbcbb5e2)</span></li>
<li>licenses/LGPL_V2 <span style="color: grey">(172c288f875590edacca050630b1c99d11eb6385)</span></li>
<li>licenses/LGPL_V3 <span style="color: grey">(fc8a5de7edf437cdc98a216370faf7c757279bcb)</span></li>
<li>licenses/QPL_V1.0 <span style="color: grey">(85bc6356f50d05096c4b877f8587e361b5fed7c7)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/114979/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>