----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129545/ -----------------------------------------------------------
Review request for Baloo. Repository: baloo Description ------- CMake: fix LMDB detection and use Diffs ----- CMakeLists.txt 299509c cmake/FindLMDB.cmake 3dc8b77 Diff: https://git.reviewboard.kde.org/r/129545/diff/ Testing ------- This fixes a compile failure found by the KDE-CI system on FreeBSD (url includes spaces): https://build-sandbox.kde.org/view/FreeBSD/job/baloo kf5-qt5 devel master FreeBSD clang/7/console 10:19:36 /home/kde/sources/kf5-qt5/baloo/src/engine/positiondb.h:29:10: fatal error: 'lmdb.h' file not found This happens because the directory in which LMDB is found is not added to the include search path; this patch fixes that. At the same time, this patch modifies the FindLMDB.cmake module to follow best-practices in writing Find modules, http://www.vtk.org/Wiki/CMake:How_To_Find_Libraries , use <foo>_INCLUDE_DIRS instead of <foo>_INCLUDE_DIR Thanks, Adriaan de Groot