Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251128
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: rc040203@freenet.de QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://corsepiu.fedorapeople.org/packages/perl-Test-Script.spec SRPM URL: http://corsepiu.fedorapeople.org/packages/perl-Test-Script-1.02-1.fc7.src.rp...
Description: The intent of this module is to provide a series of basic tests for scripts in the bin directory of your Perl distribution.
This package is a prerequisite of perl-Test-Inline. Without perl-Test-Script, upgrading/bug-fixing perl-Test-Inline is not possible.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251128
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |251110
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251128
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |249807 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
Bug 251128 depends on bug 251110, which changed state.
Bug 251110 Summary: Review Request: perl-IPC-Run3 - Run a subprocess in batch mode https://bugzilla.redhat.com/show_bug.cgi?id=251110
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
------- Additional Comments From rc040203@freenet.de 2007-09-12 05:01 EST ------- Ping? Would somebody please be so kind to review this?
It blocks me from being able to upgrade perl-Test-Inline.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fedora-perl-devel- | |list@redhat.com
------- Additional Comments From rc040203@freenet.de 2007-09-18 10:08 EST ------- Ping^2. Whom do I need to bribe to review this package?
I'd really like to be able to update perl-Test-Inline and to have it in FC-8, but without this package having been add, this won't be possible.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |tcallawa@redhat.com Status|NEW |ASSIGNED Flag| |fedora-review?
------- Additional Comments From tcallawa@redhat.com 2007-09-18 10:24 EST ------- - License tag should be "GPL+ or Artistic". - You don't need to state BuildRequires: perl >= 0:5.005, it will get pulled in by BuildRequires: perl(ExtUtils::MakeMaker) - Missing BuildRequires: perl(Test::More) (and/or perl(Test::Builder), if you'd prefer)
Make those changes, and I'll finish the review.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
------- Additional Comments From rnorwood@redhat.com 2007-09-18 11:19 EST ------- Spot beat me to it - but I'll point out that rpmlint shows that Changes and lib/Test/Script.pm are incorrectly marked as executable.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
------- Additional Comments From rc040203@freenet.de 2007-09-18 13:02 EST ------- Thanks for the feedback, updated files available at:
http://corsepiu.fedorapeople.org/packages/perl-Test-Script.spec http://corsepiu.fedorapeople.org/packages/perl-Test-Script-1.02-2.fc8.src.rp...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From tcallawa@redhat.com 2007-09-18 13:32 EST ------- Good:
- rpmlint checks return: Nothing - package meets naming guidelines - package meets packaging guidelines - license (GPL+ or Artistic) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream (8ceb782be03c0e0ce6f3b7097a327361468628b0) - package compiles on devel (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file
As usual, good work with the package Ralf. APPROVED.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From rc040203@freenet.de 2007-09-18 22:19 EST ------- New Package CVS Request ======================= Package Name: perl-Test-Script Short Description: Cross-platform basic tests for scripts Owners: corsepiu Branches: FC-6 F-7 InitialCC: fedora-perl-devel-list@redhat.com Cvsextras Commits: yes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-09-18 23:58 EST ------- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Test-Script - Cross-platform basic tests for scripts
https://bugzilla.redhat.com/show_bug.cgi?id=251128
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |1.02-2 Resolution| |CURRENTRELEASE
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=251128
Paul Howarth paul@city-fan.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |paul@city-fan.org Flag|fedora-cvs+ |fedora-cvs?
--- Comment #9 from Paul Howarth paul@city-fan.org 2012-03-14 07:52:41 EDT --- Package Change Request ====================== Package Name: perl-Test-Script New Branches: EL-5 Owners: pghmcfc InitialCC: perl-sig
Ralf has previously declared his non-interest in EPEL, which is why I am requesting this branch myself.
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=251128
Paul Howarth paul@city-fan.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |729601
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=251128
--- Comment #10 from Jon Ciesla limburgher@gmail.com 2012-03-14 08:23:55 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=251128
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2012-03-14 10:53:48 EDT --- perl-Test-Script-1.06-1.el5.1 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/perl-Test-Script-1.06-1.el5.1
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=251128
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2012-03-30 14:06:22 EDT --- perl-Test-Script-1.06-1.el5.1 has been pushed to the Fedora EPEL 5 stable repository.
package-review@lists.fedoraproject.org