<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/117579/">https://git.reviewboard.kde.org/r/117579/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/117579/diff/1/?file=266276#file266276line19" style="color: black; font-weight: bold; text-decoration: underline;">CMakeLists.txt</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">19</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="nb">set</span><span class="p">(</span><span class="s">KPEOPLE_LIBVERSION</span> <span class="s2">"${KPEOPLE_SOVERSION}.${KPEOPLE_VERSION_MINOR}.${KPEOPLE_VERSION_PATCH}"</span><span class="p">)</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">oh we still have so version number too, I misunderstood the IRC conversation.
Do we not want the major version in here too? Will save us having to change it for 1.0.0</pre>
</div>
<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;">Can you do the same for k-c-i too.</pre>
<p>- David Edmundson</p>
<br />
<p>On April 16th, 2014, 4:47 a.m. UTC, Diane Trout 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 Telepathy and David Edmundson.</div>
<div>By Diane Trout.</div>
<p style="color: grey;"><i>Updated April 16, 2014, 4:47 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
libkpeople
</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;">When submitting libkpeople to debian Sune Vuorela wanted me to adjust library version number so it starts with SONUMBER. The vast majority of shared libraries on Linux conform to the convention that libfoo.so.3 -> libfoo.so.3.4.5. The clearest documenation of the convention I could find was here: http://tldp.org/HOWTO/Program-Library-HOWTO/shared-libraries.html
I had considered using the pattern libfoo.so.3.$(package_version) AKA something like libfoo.so.3.0.2.1 but 4 digits is also a bit unusual. I decided that its ok if the major package version number and so version number differ as they are referencing different concepts package stability vs API version. However the minor and patch versions would tend to get incremented for the package and the library with roughly the same meaning (minor API compatible changes).
I can imagine 4 reasonably sane conventions.
1) library version number has nothing to do with the product version.
2) library version number is soversion+major+minor+patch.
3) library version number is soversion+minor+patch.
4) library version number is soversion+major+minor.
My first fix was #2, this version is #3. I don't really care which version works best.</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;">Applied patch. mkdir build ; cd build ; cmake .. ; find . -name "*.so" make sure the libkpeople.so.3 -> libkpeople.so.3.2.1</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/widgets/CMakeLists.txt <span style="color: grey">(8be6331)</span></li>
<li>src/CMakeLists.txt <span style="color: grey">(a04c332)</span></li>
<li>CMakeLists.txt <span style="color: grey">(5ee7c9d)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117579/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>