Review Request 117938: Support for default include path and definitions on CLang compiler.

[Luis Felipe Dominguez Vega]

On May 2, 2014, 10:26 p.m., Luis Felipe Dominguez Vega wrote:
> > Now, instead of implementing your own clangSetupStandardIncludePaths please just parameterize gccSetupStandardIncludePaths and rename that to something along 'gccLikeSetupStandardIncludePaths'. That output format won't change in the near future, you don't need to fear that. It should parse the Clang output just fine.
> > 
> > We also might want to add support for ICC (Intel Compiler) and/or others, too, at some point. Most of them are interoperable with GCC's interface and offer the same command-line options.
> 
> Kevin Funk wrote:
>     Note: Please make sure this patch actually fixes https://bugs.kde.org/show_bug.cgi?id=314218

Well i test it in my own PC purging all the gcc/g++ installation, only clang installed and the include paths are recognized from clang output... so i think that is fixed. Thanks for the recomendations....


- Luis Felipe


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


On May 2, 2014, 7:38 p.m., Luis Felipe Dominguez Vega wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117938/
> -----------------------------------------------------------
> 
> (Updated May 2, 2014, 7:38 p.m.)
> 
> 
> Review request for KDevelop and Luis Felipe Dominguez Vega.
> 
> 
> Repository: kdevelop
> 
> 
> Description
> -------
> 
> When I compile Kdevelop using only in my Debian Testing Clang 3.4, the defaults include paths is not recognized. In the code I will see that the functions are only for GCC and MSVC, with this patch now with Clang are recognized both, the include paths, and standard definitions.
> 
> 
> Diffs
> -----
> 
>   languages/cpp/CMakeLists.txt 5add7c0 
>   languages/cpp/setuphelpers.cpp 7481256 
>   languages/cpp/setuphelpers_gcc.cpp 83c6052 
>   languages/cpp/setuphelpers_gcc_like.cpp PRE-CREATION 
>   languages/cpp/tests/CMakeLists.txt dcf5087 
> 
> Diff: https://git.reviewboard.kde.org/r/117938/diff/
> 
> 
> Testing
> -------
> 
> Tested in my own PC, Debian Testing 64 bits, Clang 3.4, Kernel "Linux 3.13.10lfdominguez #1 SMP PREEMPT Sat Apr 26 17:56:25 UTC 2014 x86_64 GNU/Linux"
> 
> 
> Thanks,
> 
> Luis Felipe Dominguez Vega
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140503/886fd6f8/attachment-0001.html>