Review Request: kdevplatform: Look for ext/hash_map and unordered_map instead of checking gcc's version.

[Raphael Kubo da Costa]

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

Review request for KDevelop and Milian Wolff.


Description
-------

Follow-up to commits 3455de70d45a260607e1a4aa992a507ab979cd48 and
321e7c40bfbe838eb0a8354ff388cd88689166f9. The decision of whether to
include <ext/hash_map> or <unordered_map> for gcc/clang was based on
whether gcc > 4.3 was installed or whether clang was being used. The
latter implicitly assumed a recent enough libstdc++ version (ie. >=
4.3) was being used, which might not be the case on systems such as
FreeBSD and possibly OS X.

Instead of checking for compiler versions, we now look for these
headers: CMake first detects whether <unordered_map> is present and,
in case it is not, it looks for <ext/hash_map>. The checks in
language/editor/modificationrevision.cpp have been updated as
well. This should cover all the cases being previously detected, as
well as fix the checks for FreeBSD and other systems where the build
was failing before with clang.


Diffs
-----

  CMakeLists.txt 365e00f 
  config-kdevplatform.h.cmake 6737e06 
  language/editor/modificationrevision.cpp ac8a852 

Diff: http://git.reviewboard.kde.org/r/105066/diff/


Testing
-------

kdevplatform built fine with gcc 4.2.1 and clang + libstdc++ from gcc 4.2.1. It'd be good for someone on Linux to give this a try with a more recent libstdc++ (or LLVM's libc++).


Thanks,

Raphael Kubo da Costa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120527/77b327df/attachment.html>