[Kde-pim] Review Request 115318: Fix strigi analyzers not compiling even though strigi is found

Matthew John Dawson matthew at mjdsystems.ca
Mon Jan 27 16:46:50 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115318/
-----------------------------------------------------------

(Updated Jan. 27, 2014, 11:46 a.m.)


Review request for KDEPIM.


Changes
-------

Remove master from the description.  It isn't relevant anymore.


Repository: kdepim


Description (updated)
-------

During Plasma Active work, I discovered the strigi analyzers were no longer compiling or installing.  At some point, the variable for whether strigi is found or not changed its name to be all uppercase.  Fix the top level CMakeLists.txt to account for this difference.

I have no idea when this change occurred, or why it wasn't noticed earlier.

This patch is originally from KDE/4.11, forwarded ported to KDE/4.12.  I'd like to commit to 4.11 and 4.12.


Diffs
-----

  CMakeLists.txt b445e3a 

Diff: https://git.reviewboard.kde.org/r/115318/diff/


Testing
-------

Just a straight compile was tested on the KDE/4.11 branch, were the plasma active work was being done.


Thanks,

Matthew John Dawson

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list