<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/110898/">http://git.reviewboard.kde.org/r/110898/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 9th, 2013, 10:27 a.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Being past the Feature Freeze doesn't seem a good idea to me to remove a feature.</pre>
</blockquote>
<p>On June 9th, 2013, 4:11 p.m. UTC, <b>Simeon Bird</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">It isn't really removing a feature - the code in question does nothing at all that I can see, and hasn't done for a while. In fact, I'm not sure the FontAnalyzer really did anything even when strigi was in use, because /usr/share/fonts was not indexed by default. Its just removing dead code.
It *is* a dependency change though, and I'm not sure what the rules are for that. I guess it could be made optional, but since it doesn't do anything that doesn't seem really worth the effort.</pre>
</blockquote>
<p>On June 9th, 2013, 8:07 p.m. UTC, <b>Vishesh Handa</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The FontAnalyzer has never been used by Nepomuk. That does not mean that one one has ever used it.
Maybe someone installs KDE and uses Strigi directly and therefore uses the Font Analyzer? Or maybe someone uses KFileMetaInfo, which uses Strigi internally. It doesn't seem like anyone is - http://lxr.kde.org/ident?i=KFileMetaInfo - but this is just the code in the KDE repositories.
Do we want to risk breaking existing code even if it isn't ours? Considering that Strigi is still going to be a compile time dependency for kdelibs, I don't really see the big deal in having it for kde-workspace.</pre>
</blockquote>
<p>On June 9th, 2013, 9:07 p.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Vishesh: exactly what i wanted to say, i just was lazy to write it all in clear.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">hm, yes, ok. You're quite right, I'll ditch the patch.</pre>
<br />
<p>- Simeon</p>
<br />
<p>On June 10th, 2013, 1:09 a.m. UTC, Simeon Bird 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-workspace and Nepomuk.</div>
<div>By Simeon Bird.</div>
<p style="color: grey;"><i>Updated June 10, 2013, 1:09 a.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;">So far as I can see, the only thing depending on strigi still in kde-workspace is a strigi plugin called FontAnalyzer,
and, I think, this doesn't do much now nepomuk doesn't use strigi. (I'm curious as to why it was in kde-workspace
in the first place, instead of in strigi, actually)
This patch removes the strigi dependency and FontAnalyzer from kde-workspace.</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;">Compiled, ran</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>kcontrol/kfontinst/CMakeLists.txt <span style="color: grey">(1ba461cf9c09a0dba3af13b67f585db08c7da77a)</span></li>
<li>CMakeLists.txt <span style="color: grey">(d233cf4a745ea1c6caf0b560539a68f71ab41020)</span></li>
<li>kcontrol/kfontinst/strigi-analyzer/CMakeLists.txt <span style="color: grey">(ab5643ec4f1ce47f17b598e016300d51e181c946)</span></li>
<li>kcontrol/kfontinst/strigi-analyzer/FontEngine.h <span style="color: grey">(028e971f34a52b4bda4df8130cbe2af688c45e8b)</span></li>
<li>kcontrol/kfontinst/strigi-analyzer/FontEngine.cpp <span style="color: grey">(8e00bd550c9e2a3e1b25ef6cf6a3d510576e67d8)</span></li>
<li>kcontrol/kfontinst/strigi-analyzer/FontThroughAnalyzer.h <span style="color: grey">(0e0a7a8eac99c85ed21d9ab3305a2a75b18e9ec9)</span></li>
<li>kcontrol/kfontinst/strigi-analyzer/FontThroughAnalyzer.cpp <span style="color: grey">(ffa22a9ad09617b6ab87652ee0a9f246e0502227)</span></li>
<li>plasma/desktop/applets/kickoff/CMakeLists.txt <span style="color: grey">(e9e288866ecdcfd7aa35c086dfe78d4252748995)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110898/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>