Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: perl-ExtUtils-InstallPAR - Install .par's into any installed perl
https://bugzilla.redhat.com/show_bug.cgi?id=498191
Summary: Review Request: perl-ExtUtils-InstallPAR - Install .par's into any installed perl Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mmaslano@redhat.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: ---
Spec URL: http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InstallPAR.spec SRPM URL: http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InstallPAR-0.03-1.fc11... Description: This module installs PAR distributions (i.e. .par files) into any perl installation on the system. The PAR::Dist module can install into the currently running perl by default and provides the necessary parameters to override any installation directories. Figuring out how to use those overrides in order to install into an arbitrary perl installation on the system may be beyond most users, however. Hence this convenience wrapper using ExtUtils::InferConfig to automatically determine the typical site installation paths of any perl interpreter than can be executed by the current user.
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=498191
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |panemade@gmail.com AssignedTo|nobody@fedoraproject.org |panemade@gmail.com Flag| |fedora-review?
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=498191
--- Comment #1 from Parag AN(पराग) panemade@gmail.com 2009-04-30 00:37:25 EDT ---
koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1329657 Build failed. Need BR: perl(Test::More)
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=498191
--- Comment #2 from Marcela Maslanova mmaslano@redhat.com 2009-04-30 02:19:02 EDT --- http://koji.fedoraproject.org/koji/taskinfo?taskID=1329831 http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InstallPAR-0.03-2.fc11...
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=498191
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #3 from Parag AN(पराग) panemade@gmail.com 2009-04-30 04:40:20 EDT --- Review: + package builds in mock (rawhide i586). koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1329831 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 88832cdff670a78bb44e25cadd700f4d ExtUtils-InstallPAR-0.03.tar.gz
+ package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=3, Tests=1, 0 wallclock secs ( 0.03 usr 0.00 sys + 0.07 cusr 0.01 csys = 0.11 CPU) + Package perl-ExtUtils-InstallPAR-0.03-2.fc12.noarch => Provides: perl(ExtUtils::InstallPAR) = 0.03 Requires: perl(Carp) perl(Config) perl(Exporter) perl(File::Spec) perl(PAR::Dist) perl(strict) perl(vars) + Not a GUI application
APPROVED.
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=498191
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #4 from Marcela Maslanova mmaslano@redhat.com 2009-04-30 04:53:07 EDT --- New Package CVS Request ======================= Package Name: perl-ExtUtils-InstallPAR Short Description: Install .par's into any installed perl Owners: mmaslano Branches: F-11 InitialCC: perl-sig
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=498191
Dennis Gilmore dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #5 from Dennis Gilmore dennis@ausil.us 2009-05-01 17:00:48 EDT --- CVS Done
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=498191
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |CURRENTRELEASE
package-review@lists.fedoraproject.org