[Kde-pim] Review Request 127074: Make stdlib exception include detection work with clang/libc++

[Armin K.]

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

(Updated Feb. 15, 2016, 5:36 p.m.)


Status
------

This change has been marked as submitted.


Review request for Akonadi and Daniel Vrátil.


Changes
-------

Submitted with commit 036031f520ef1f5bdc40f9f9e8a2951be311eb3b by Armin K to branch master.


Repository: akonadi


Description
-------

Currently, akonadi has some code to detect full path to C++ exception include.

This code doesn't work if -DCMAKE_CXX_COMPILER is set to "clang++ -stdlib=libc++", becaue cmake strips the second parameter, only leaving full path to the compiler.

It doesn't even work when I set -DCMAKE_CXX_FLAGS to "-stdlib=libc++" as the cmake module that looks for the include path doesn't use them. Even if I try to use them, clang chokes because there are some flags that shouldn't be passed in combination with -stdlib=libc++

The only sane path is this. If -DCXX_STDLIB is passed to cmake (ie, -DCXX_STDLIB=libc++), the cmake scriptlet only uses that flag. If nothing's passed to the variable, it's ignored.


Diffs
-----

  CMakeLists.txt 05a94d3 
  cmake/modules/FindStdlibInclude.cmake 135e655 

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


Testing
-------

Now builds with both clang++ -stdlib=libc++ and plain ol' g++


Thanks,

Armin K.

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