Bug 571282
Summary: | Review Request: perl-CatalystX-LeakChecker - Debug memory leaks in Catalyst applications | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Chris Weyl <cweyl> |
Component: | Package Review | Assignee: | Steve Traylen <steve.traylen> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, notting, steve.traylen |
Target Milestone: | --- | Flags: | steve.traylen:
fedora-review+
tcallawa: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://search.cpan.org/dist/CatalystX-LeakChecker | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-03-16 04:34:39 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Chris Weyl
2010-03-08 01:26:58 UTC
Review: perl-CatalystX-LeakChecker-0.05-1.fc12.src.rpm Date: March 11th 2010 Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2047403 * PASS: rpmlint output rpmlint SRPMS/perl-CatalystX-LeakChecker-0.05-1.fc14.src.rpm \ SPECS/perl-CatalystX-LeakChecker.spec \ RPMS/noarch/perl-CatalystX-LeakChecker-0.05-1.fc14.noarch.rpm \ RPMS/noarch/perl-CatalystX-LeakChecker-tests-0.05-1.fc14.noarch.rpm perl-CatalystX-LeakChecker-tests.noarch: W: no-documentation 3 packages and 1 specfiles checked; 0 errors, 1 warnings. * PASS: Named according to the Package Naming Guidelines. * PASS: spec file name same as base package %{name}. * PASS: Packaging Guidelines. * PASS: Approved license in .spec file. GPL & Artist. * PASS: License on Source code. * PASS: Include LICENSE file or similar if it exist. LICENSE include. * PASS: Written in American English. * PASS: Spec file legible. * PASS: Included source must match upstream source. c6227507da51072f2ef34ea164346bf3 CatalystX-LeakChecker-0.05.tar.gz c6227507da51072f2ef34ea164346bf3 ../SOURCES/CatalystX-LeakChecker-0.05.tar.gz * PASS: Build on one architecture. * PASS: Not building on an architecture must highlighted. is no arch. * PASS: Build dependencies must be listed in BuildRequires. * PASS: Handle locales properly. No locales. * PASS: ldconfig must be called on shared libs. No libs * PASS: No bundled copies of system libraries. * PASS: Package must state why relocatable if relocatable. Not relocatable. * PASS: A package must own all directories that it creates * PASS: No duplicate files in %files listings. * PASS: Permissions on files must be set properly. %defattr * PASS: %clean section contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). * PASS: Each package must consistently use macros. * PASS: The package must contain code, or permissable content. * PASS: Large documentation files must go in a -doc subpackage. No large docs. * PASS: %doc must not affect the runtime of the application. * PASS: Header files must be in a -devel package. No headers. * PASS: Static libraries must be in a -static package. No statics. * PASS: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' * PASS: Then library files that end in .so (without suffix) must go in a -devel package. * PASS: devel packages must require the exact base package * PASS: No .la libtool archives * PASS: GUI apps should have %{name}.desktop file * PASS: No files or directories already owned by other packages. * PASS: %install must run rm -rf %{buildroot} (or $RPM_BUILD_ROOT). * PASS: All filenames in rpm packages must be valid UTF-8. Just one question What is '%{?perl_ext_env_unset}' on my system it is unset. Whoops -- sorry about that, %perl_ext_env_unset is a macro I have defined on my system; I'm thinking about including it in the macros.perl file. Basically, it unsets any "special" env vars that happen to be set, as well as sets some to get some of the more interactive Makefile.PL's to go with the defaults: %perl_ext_env_unset %{expand: unset PERL_MM_OPT MODULEBUILDRC PERL5INC export PERL_AUTOINSTALL="--defaultdeps" export PERL_MM_USE_DEFAULT=1 } It's conditionalized in this package, so it does no harm, but I can remove it if you'd prefer. I guess it depends how likely it is perl_ext_env_unset is to become standard. Anyway I'll approve and you decide. I'm thinking the perl guidelines could do with an update of many of these new macros. Would be good to keep the old methods in there though as options for the benefit of EPEL. Steve New Package CVS Request ======================= Package Name: perl-CatalystX-LeakChecker Short Description: Debug memory leaks in Catalyst applications Owners: cweyl Branches: F-11 F-12 F-13 devel InitialCC: perl-sig CVS done (by process-cvs-requests.py). Thanks for the review! :-) |