[Bug 927310] Review Request: libgit2 - The Git linkable library

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=927310

Michael Schwendt <mschwendt@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mschwendt@xxxxxxxxx

--- Comment #1 from Michael Schwendt <mschwendt@xxxxxxxxx> ---
It's surprising how you've mispackaged this library. :-(

> License:       GPL2 with linking exemption.

As a valid tag, at most there is "License: GPLv2 with exceptions" in the Fedora
Licensing guidelines: https://fedoraproject.org/wiki/Licensing:Main

> pure C implementation

If it's written in C, a working -debuginfo package ought to be built.

> # Cleanup
> rm -rf %{buildroot}/usr/lib/debug %{buildroot}/usr/src/debug

https://fedoraproject.org/wiki/Packaging:Guidelines#Debuginfo_packages

> Requires:	%{name} = %{version}-%{release}

https://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

> %package 	devel
> Summary:	The Git linkable library

This is half-hearted summary for the -devel package. That package does not
contain the library, but just the files needed for building software that uses
libgit2.

> %description 	devel
> libgit2 is a portable, pure C implementation of the 
> Git core methods provided as a re-entrant linkable 
> library with a solid API, allowing you to write native 
> speed custom Git applications in any language with bindings.

Same here. Just copying the %description of the base package is misleading.

> /usr/lib/%{name}*

And what about target platforms where %{_libdir} is /usr/lib64?

> %files devel
> /usr/include/git2*
> /usr/lib/pkgconfig

Same here. And since the package cannot be built yet, I assume one of the
library files is misplaced, too:
https://fedoraproject.org/wiki/Packaging:Guidelines#Devel_Packages

A simple test-build for Fedora 18 fails very early, unfortunately.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2O6KdWkkCd&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]