<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Feb 25, 2015 at 6:33 PM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wednesday 25 February 2015 15:38:19 Aleix Pol wrote:<br>
> At the moment KPeople is an optional dependency, there's still the<br>
> possibility to use it without KPeople.<br>
<br>
</span>I guess you mean s/KPeople/Baloo/g here.<br></blockquote><div><br></div><div>No this was actually KPeople being optional dep of KTp, but the truth is that</div><div>all the code is actually only tested with KPeople as that's pretty much what</div><div>we're aiming for in the past couple years (but KPeople went through many</div><div>many changes, it started as Nepomuk thing, then akonadi, now it's self-contained,</div><div>so we never wanted to hard-depend on it). Now we've optimized everything</div><div>for KPeople usage so imho it makes only sense to not discard it now and</div><div>in fact make it finally a hard-dependency of KDE Telepathy.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The problem is: Baloo depends on a few frameworks,<br>
and now we would have KPeople depending on Baloo.<br>
So it wouldn't be possible to build all of frameworks together, one would need<br>
to interject Baloo in the middle of it, in order to be able to build "with all<br>
options" i.e. with this optional dependency.<br>
<br>
The reason "it's optional so it doesn't matter" means you might as well delete<br>
the code, since it becomes impractical to build it anyway.<br>
More seriously I would suggest moving the plugin to Baloo maybe, if it makes<br>
sense for it to depend on KPeople, or moving it in workspace maybe (but then<br>
it's not available in other workspaces). Or to this new "product" (set of<br>
modules) that I think we should have to host drkonqi, kio-extras etc., i.e.<br>
the set of runtime deps common to workspace and applications. This issue keeps<br>
popping up.<br>
<br>
If you want a short term solution, I'd say the plugin should stay in<br>
playground somewhere.<br>
I don't like the optional dependency thing because it will still show up in<br>
dependency diagrams, making a mess with baloo being in the middle of<br>
frameworks while it was decided that it's not.<br></blockquote><div><br></div><div>As I said it's not even being used right now, so imho would be fine if it got</div><div>removed/disabled altogether.</div></div><div><br></div><div>Cheers</div>-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div></div>
</div></div>