[Kde-pim] Review Request 126601: Use Sasl2_INCLUDE_DIRS in kioslaves

[Tobias Berner]

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

(Updated Jan. 4, 2016, 8:32 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDEPIM-Libraries.


Changes
-------

Submitted with commit 50cb6dea28f4ba86df823a8af9b2c037e05990c8 by Daniel Vrátil on behalf of Tobias Berner to branch Applications/15.12.


Repository: kdepimlibs


Description
-------

The cmake file kioslave/cmake/FindSasl2.cmake defines the
Sasl2 include path in Sasl2_INCLUDE_DIRS.
But
 * kioslaves/src/smtp/CMakeLists.txt
 * kioslave/src/smtp/tests/CMakeLists.txt
 * kioslaves/src/pop3/CMakeLists.txt
used the following
| [...] ${Sasl2_INCLUDE_DIR} [...]
Note the missing "S" at the end.

This leads to the following build error on FreeBSD
|[ 94%] Building CXX object kioslave/src/smtp/CMakeFiles/kio_smtp.dir/smtp.cpp.o
|/tmp/usr/ports/deskutils/kdepimlibs-kf5/work/kdepimlibs-15.12.0/kioslave/src/smtp/smtp.cpp:35:10: fatal error:
|      'sasl/sasl.h' file not found
|#include <sasl/sasl.h>
|         ^
|1 error generated.

This patch uses the correct Sasl2_INCLUDE_DIRS fixing this problem.


Diffs
-----

  kioslave/src/pop3/CMakeLists.txt 6433873 
  kioslave/src/smtp/CMakeLists.txt 9aba907 
  kioslave/src/smtp/tests/CMakeLists.txt d2d25db 

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


Testing
-------

build locally


Thanks,

Tobias Berner

_______________________________________________
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/