Review Request 110898: Remove unused strigi dependency from tree, along with unused strigi FontAnalyzer.
Albert Astals Cid
aacid at kde.org
Sun Jun 9 22:06:59 BST 2013
> On June 9, 2013, 10:27 a.m., Albert Astals Cid wrote:
> > Being past the Feature Freeze doesn't seem a good idea to me to remove a feature.
>
> Simeon Bird wrote:
> 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.
>
> Vishesh Handa wrote:
> 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.
Vishesh: exactly what i wanted to say, i just was lazy to write it all in clear.
- Albert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110898/#review33978
-----------------------------------------------------------
On June 9, 2013, 1:53 a.m., Simeon Bird wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110898/
> -----------------------------------------------------------
>
> (Updated June 9, 2013, 1:53 a.m.)
>
>
> Review request for kde-workspace and Nepomuk.
>
>
> Description
> -------
>
> 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.
>
>
> Diffs
> -----
>
> kcontrol/kfontinst/CMakeLists.txt 1ba461cf9c09a0dba3af13b67f585db08c7da77a
> CMakeLists.txt d233cf4a745ea1c6caf0b560539a68f71ab41020
> kcontrol/kfontinst/strigi-analyzer/CMakeLists.txt ab5643ec4f1ce47f17b598e016300d51e181c946
> kcontrol/kfontinst/strigi-analyzer/FontEngine.h 028e971f34a52b4bda4df8130cbe2af688c45e8b
> kcontrol/kfontinst/strigi-analyzer/FontEngine.cpp 8e00bd550c9e2a3e1b25ef6cf6a3d510576e67d8
> kcontrol/kfontinst/strigi-analyzer/FontThroughAnalyzer.h 0e0a7a8eac99c85ed21d9ab3305a2a75b18e9ec9
> kcontrol/kfontinst/strigi-analyzer/FontThroughAnalyzer.cpp ffa22a9ad09617b6ab87652ee0a9f246e0502227
> plasma/desktop/applets/kickoff/CMakeLists.txt e9e288866ecdcfd7aa35c086dfe78d4252748995
>
> Diff: http://git.reviewboard.kde.org/r/110898/diff/
>
>
> Testing
> -------
>
> Compiled, ran
>
>
> Thanks,
>
> Simeon Bird
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130609/04568c05/attachment.htm>
More information about the kde-core-devel
mailing list