Review Request 110898: Remove unused strigi dependency from tree, along with unused strigi FontAnalyzer.

Hrvoje Senjan hrvoje.senjan at gmail.com
Sun Jun 9 19:42:54 BST 2013



> On June 9, 2013, 8:38 a.m., Christophe Giboudeaux wrote:
> > plasma/desktop/applets/kickoff/core/config-kickoff-applets.h.cmake:#cmakedefine HAVE_STRIGIDBUS 1
> > 
> > this line is not needed anymore with this change
> 
> Simeon Bird wrote:
>     Thanks - that file consists only of that line. Should I also remove the file and the line in plasma/desktop/applets/kickoff/CMakeLists.txt that refers to it, or is it needed for some other reason?

There is also already opened review for kickoff part https://git.reviewboard.kde.org/r/110090/
It has a ship it, but looks it's not commited.


- Hrvoje


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110898/#review33969
-----------------------------------------------------------


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/12ee8eba/attachment.htm>


More information about the kde-core-devel mailing list