Review Request 115331: Make kdev-clang compile with llvm-3.4

Kevin Funk krf at gmx.de
Mon Jan 27 18:54:32 UTC 2014


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


Just discussed a bit with Aleix: While the patch isn't perfect, let's just commit this.

I'll try to come up with a better solution in the meantime.

- Kevin Funk


On Jan. 27, 2014, 4:13 p.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115331/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2014, 4:13 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-clang
> 
> 
> Description
> -------
> 
> Currently FindLLVM.cmake looking for llvm-config-3.3 and llvm-config only. But on my machine there is only llvm-config-3.4. 
> To bypass it let's try to find llvm-clang instead, then retrieve it's directory and try to find llvm-config-* in it.
> 
> NOTE: I'm not very good at cmake, so it'll be nice if someone more experienced could tell if there is a more simple/correct way to achieve it.
> 
> 
> Diffs
> -----
> 
>   cmake/FindLLVM.cmake b77528f 
> 
> Diff: https://git.reviewboard.kde.org/r/115331/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

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


More information about the KDevelop-devel mailing list