diff options
| author | Alexander Neundorf <neundorf@kde.org> | 2008-11-13 00:16:49 +0000 |
|---|---|---|
| committer | Alexander Neundorf <neundorf@kde.org> | 2008-11-13 00:16:49 +0000 |
| commit | 4a2b41ff7cd32887f7af33b9078eb7dad567fa8c (patch) | |
| tree | b0b4b3407b007407e1c7a3b5b001e2575f190b47 /modules/FindFFmpeg.cmake | |
| parent | f351181f55c94cba3988ef6be223963ec297b88a (diff) | |
| download | extra-cmake-modules-4a2b41ff7cd32887f7af33b9078eb7dad567fa8c.tar.gz extra-cmake-modules-4a2b41ff7cd32887f7af33b9078eb7dad567fa8c.tar.bz2 | |
-fix FindStrigi when pkg-config didn't find strigi:
in this case STRIGI_INCLUDEDIR is empty, and then used as result
variable for FIND_PATH(). But FIND_PATH() only does something if
the used variable contains "NOTFOUND", so it didn't search for the header actually.
Now FIND_PATH() uses STRIGI_INCLUDE_DIR and sets STRIGI_INCLUDEDIR for compatibility.
Hint: FOO_INCLUDE_DIR is the correct naming according to the cmake module coding rules
(see kdelibs/cmake/modules/cmake-modules/styleguide.txt), not legacy.
Alex
CCMAIL: helio@kde.org
Helio: are you sure there are not other such breakages left ? I'm always *very* cautious when touching these files,
I still have a bad feeling after these big changes.
svn path=/trunk/KDE/kdelibs/; revision=883470
Diffstat (limited to 'modules/FindFFmpeg.cmake')
0 files changed, 0 insertions, 0 deletions