[Bug 620177] Review Request: uprof - Profiling toolkit
bugzilla at redhat.com
bugzilla at redhat.com
Mon Aug 2 14:49:52 UTC 2010
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=620177
Kalev Lember <kalev at smartlink.ee> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
CC| |kalev at smartlink.ee
AssignedTo|nobody at fedoraproject.org |kalev at smartlink.ee
Flag| |fedora-review?
--- Comment #1 from Kalev Lember <kalev at smartlink.ee> 2010-08-02 10:49:51 EDT ---
Taking for review.
> License: GPLv2+
How did you determine that it is GPLv2+? uprof.c, uprof.h, and uprof-private.h
all have LGPLv2+ headers and there's an LGPL COPYING file too.
> find $RPM_BUILD_ROOT -name '*.a' -exec rm -f {} ';'
It'd be nicer to pass --disable-static to configure so that the .a files don't
get built in the first place.
> BuildRequires: gtk-doc
> BuildRequires: gnome-doc-utils
Are these buildrequires necessary? There doesn't appear to be anything built
with gtk-doc in the final rpms.
The COPYING file is included in both base package and in the -devel subpackage.
As per
https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Subpackage_Licensing
you can omit the COPYING file in -devel as it's dependant upon the base
package.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the package-review
mailing list