<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="http://git.reviewboard.kde.org/r/112583/">http://git.reviewboard.kde.org/r/112583/</a>
</td>
</tr>
</table>
<br />
<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="http://git.reviewboard.kde.org/r/112583/diff/1/?file=188103#file188103line19" style="color: black; font-weight: bold; text-decoration: underline;">staging/kconfigwidgets/src/CMakeLists.txt</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">19</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="s">KCodecs</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">19</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="o"><span class="hl">${</span></span><span class="nv">KCodecs<span class="hl">_LIBRARIES</span></span><span class="o"><span class="hl">}</span></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;">AFAIK, this is not needed. When imported, the *Config.cmake files create something that is called "imported targets" in cmake which basically means that you can use it right away.
It's similar to how you can link against QtCore using Qt5::Core and you don't need a ${variable}.
BTW, are you sure that many publicly linked libraries are needed?</pre>
</div>
<br />
<p>- Aleix</p>
<br />
<p>On September 7th, 2013, 7:45 p.m. UTC, David Edmundson wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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.</div>
<div>By David Edmundson.</div>
<p style="color: grey;"><i>Updated Sept. 7, 2013, 7:45 p.m.</i></p>
<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;">KConfigWidgets relies on a private header in KWidgetAddons. This prevents KConfigWidgets building on it's own
By exposing the two of these private members in protected methods, we can solve this problem.
Commit log below:
--------------
change target link libraries to ${lib_LIBRARIES} format
--
Remove inheritance on KSelectionActionPrivate in KRecentFileActions
This is in a different framework so cannot be used when building
standalone.
--
Expose associated comboboxes and toolbuttons in KSelectAction</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>staging/kconfigwidgets/src/CMakeLists.txt <span style="color: grey">(dad6bcc140cb4fe56380bd44ea007fe1f06edace)</span></li>
<li>staging/kconfigwidgets/src/krecentfilesaction.h <span style="color: grey">(edf77a24866018134d9c5a200c6e35213742c555)</span></li>
<li>staging/kconfigwidgets/src/krecentfilesaction.cpp <span style="color: grey">(9aa5f8bd83d064402dd6333c7cd89a388d373fe1)</span></li>
<li>staging/kconfigwidgets/src/krecentfilesaction_p.h <span style="color: grey">(8b3cc8f82b248cc9ed8a1f1be69cf4160f97378c)</span></li>
<li>tier1/kwidgetsaddons/src/kselectaction.h <span style="color: grey">(07afaa960d2be4877aeef0ff074d0fd20c7b7f40)</span></li>
<li>tier1/kwidgetsaddons/src/kselectaction.cpp <span style="color: grey">(b388df94baa042e8230f2795f35dd378117c6ab6)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/112583/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>