Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Package review: perl-Probe-Perl
https://bugzilla.redhat.com/show_bug.cgi?id=463771
Summary: Package review: perl-Probe-Perl Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium 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
description This module provides methods for obtaining information about the currently running perl interpreter. It originally began life as code in the Module::Build project, but has been externalized here for general use.
http://mmaslano.fedorapeople.org/perl-Probe-Perl/perl-Probe-Perl.spec http://mmaslano.fedorapeople.org/perl-Probe-Perl/perl-Probe-Perl-0.01-1.fc9....
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=463771
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |457517
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=463771
Dan Horák dan@danny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |dan@danny.cz AssignedTo|nobody@fedoraproject.org |dan@danny.cz Flag| |fedora-review+
--- Comment #1 from Dan Horák dan@danny.cz 2008-09-25 02:57:04 EDT --- formal review is here, see the notes below:
OK source files match upstream: 36a9dde1953c0ab0bcd3998fca876b8f20f245cd Probe-Perl-0.01.tar.gz OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK build root is correct. OK license field matches the actual license. OK license is open source-compatible. License text not included upstream. OK latest version is being packaged. OK BuildRequires are proper. N/A compiler flags are appropriate. OK %clean is present. OK package builds in mock (Rawhide/x86_64). N/A debuginfo package looks complete. OK rpmlint is silent. OK* final provides and requires look sane. OK %check is present and all tests pass. OK no shared libraries are added to the regular linker search paths. OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK no scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK no headers. OK no pkgconfig files. OK no libtool .la droppings. OK not a GUI app.
- the explicit R: perl(Config) is redundant, it is found automagically
this package is 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=463771
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #2 from Marcela Maslanova mmaslano@redhat.com 2008-09-25 04:30:03 EDT --- I'll remove perl(Config) for CVS build.
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=463771
--- Comment #3 from Marcela Maslanova mmaslano@redhat.com 2008-09-25 04:31:49 EDT --- New Package CVS Request ======================= Package Name: perl-Probe-Perl Short Description: obtain information about perl interpreter Owners: mmaslano@redhat.com Branches: rawhide InitialCC: fedora-perl-devel-list@redhat.com
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=463771
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #4 from Kevin Fenzi kevin@tummy.com 2008-09-28 15:01:04 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=463771
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
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=463771
Paul Howarth paul@city-fan.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |paul@city-fan.org
--- Comment #5 from Paul Howarth paul@city-fan.org 2012-03-14 07:43:16 EDT --- Marcela, could I please have an EPEL-5 branch for this package?
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=463771
Marcela Mašláňová mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #6 from Marcela Mašláňová mmaslano@redhat.com 2012-03-14 09:10:58 EDT --- New Package CVS Request ======================= Package Name: perl-Probe-Perl Short Description: obtain information about perl interpreter Owners: pghmcfc Branches: EL-5 InitialCC: perl-devel
[Paul asked me for a new branch for EL-5.]
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=463771
--- Comment #7 from Jon Ciesla limburgher@gmail.com 2012-03-14 09:15:17 EDT --- Already exists, use a Package Change request to add a branch. And use perl-sig, not perl-devel.
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=463771
Paul Howarth paul@city-fan.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #8 from Paul Howarth paul@city-fan.org 2012-03-14 09:36:08 EDT --- Package Change Request ====================== Package Name: perl-Probe-Perl New Branches: EL-5 Owners: pghmcfc 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=463771
--- Comment #9 from Jon Ciesla limburgher@gmail.com 2012-03-14 09:40:00 EDT --- Git done (by process-git-requests).
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=463771
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2012-03-14 10:05:41 EDT --- perl-Probe-Perl-0.01-4.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/perl-Probe-Perl-0.01-4.el5
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=463771
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2012-03-30 14:04:26 EDT --- perl-Probe-Perl-0.01-4.el5 has been pushed to the Fedora EPEL 5 stable repository.
package-review@lists.fedoraproject.org