Review Request 120196: Find-Module to detect libgit2

Christoph Cullmann cullmann at kde.org
Mon Sep 15 18:31:16 UTC 2014



> 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.
> 
> Christoph Cullmann wrote:
>     I can rename it, but then it is inconsistent with most other FindXXX.cmake's, as they skip the lib prefix.
> 
> Aleix Pol Gonzalez wrote:
>     I'm unsure it's inconsistent. The project is actually called libgit2, because it's a library that does git things: https://libgit2.github.com/
>     
>     As an example, here's archlinux:
>     https://www.archlinux.org/packages/extra/i686/git/
>     https://www.archlinux.org/packages/community/x86_64/libgit2/
>     
>     And in general, archlinux doesn't prefix the lib for frameworks anyway:
>     https://www.archlinux.org/packages/extra/x86_64/kcoreaddons/
>     
>     Does that make sense? :)
> 
> Christoph Cullmann wrote:
>     Ok, that seems to be reasonable, will change that, if Alex is ok with that, too ;=)
>     Other remarks?
> 
> Christoph Cullmann wrote:
>     Btw., that rename would imply that the vars are LIBGIT2_... and the Target would be LibGit2::LibGit2, or?
> 
> Aleix Pol Gonzalez wrote:
>     Yes, +1 to that.

Renamed, its ok?
Need to retest it again, but in principle should work.


- Christoph


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


On Sept. 15, 2014, 6:30 p.m., Christoph Cullmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120196/
> -----------------------------------------------------------
> 
> (Updated Sept. 15, 2014, 6:30 p.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/FindLibGit2.rst PRE-CREATION 
>   find-modules/FindLibGit2.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/4fd71b53/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list