Review Request 120196: Find-Module to detect libgit2

[Christoph Cullmann]

> On Sept. 15, 2014, 3:15 p.m., Aleix Pol Gonzalez wrote:
> > Shouldn't it be called FindLibGit2?
> > 
> > Git2 and LibGit2 seem different things to me.

I can rename it, but then it is inconsistent with most other FindXXX.cmake's, as they skip the lib prefix.


- Christoph


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


On Sept. 14, 2014, 10:19 a.m., Christoph Cullmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120196/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2014, 10:19 a.m.)
> 
> 
> Review request for KDE Frameworks and Alex Merry.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Add a find module that allows to detect libgit2.
> Will be used by KTextEditor & Kate.
> Will remove the copy in KTextEditor after this is merged.
> 
> 
> Diffs
> -----
> 
>   docs/find-module/FindGIT2.rst PRE-CREATION 
>   find-modules/FindGIT2.cmake PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/120196/diff/
> 
> 
> Testing
> -------
> 
> Tested with KTextEditor, version detection is here important, as libgit2 did change API a lot, seems to work on Linux, hope it works on Windows, too.
> 
> 
> Thanks,
> 
> Christoph Cullmann
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140915/7916aadf/attachment.html>