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/show_bug.cgi?id=450553
Summary: Review Request: perl-TAP-Harness - Run Perl standard test scripts with statistics Product: Fedora Version: rawhide Platform: All URL: http://v3.sk/~lkundrak/SPECS/perl-TAP-Harness.spec OS/Version: Linux Status: NEW Severity: low Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: lkundrak@v3.sk QAContact: extras-qa@fedoraproject.org CC: fedora-package- review@redhat.com,notting@redhat.com,tcallawa@redhat.com
SPEC: http://v3.sk/~lkundrak/SPECS/perl-TAP-Harness.spec SRPM: http://v3.sk/~lkundrak/mock-results/perl-TAP-Harness-3.10-1.el5.noarch/perl-... mock: http://v3.sk/~lkundrak/mock-results/perl-TAP-Harness-3.10-1.el5.noarch/
Description:
Run Perl tests and output formatted results with statistics.
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
------- Additional Comments From lkundrak@v3.sk 2008-06-09 11:40 EST ------- Note that this is perl Test-Harness 3.10 package, stripped of Test::Harness compatibility module, because older version of Test::Harness is included in perl package (and perl-Test-Harness since Fedora 9).
This means that packages that rely on Test::Harness interface won't be affected by this package, and only TAP::Harness is provided by this package.
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
------- Additional Comments From rc040203@freenet.de 2008-06-09 11:46 EST ------- I am very strongly opposed to letting this package in and am leaning to reject it.
What is the issue you are trying to solve?
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
------- Additional Comments From lkundrak@v3.sk 2008-06-09 12:46 EST ------- Ralf: The issue I am trying to solve is lack of TAP::Harness module in Fedora. Could you please name specific reasons against letting the package in Fedora?
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mmahut@redhat.com Flag| |fedora-review+
------- Additional Comments From mmahut@redhat.com 2008-06-10 06:49 EST ------- I don't see anything wrong with this package, it looks sane. 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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From lkundrak@v3.sk 2008-06-10 07:13 EST ------- New Package CVS Request ======================= Package Name: perl-TAP-Harness Short Description: Run Perl standard test scripts with statistics Owners: lkundrak Branches: F-8 F-9 EL-4 EL-5 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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs-
------- Additional Comments From rc040203@freenet.de 2008-06-10 08:05 EST ------- (In reply to comment #4)
I don't see anything wrong with this package, it looks sane. approved.
I do see many issues
a) This package is not a CPAN distro, it's a submodule of Test::Harness b) This package doesn't install correctly - It wants to replace Test::Harness c) This package will collide with a future upgrade of Test::Harness
You are outsmarting yourselves.
DISAPPROVING.
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review+ |fedora-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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
------- Additional Comments From lkundrak@v3.sk 2008-06-10 09:04 EST ------- (In reply to comment #6)
(In reply to comment #4)
I don't see anything wrong with this package, it looks sane. approved.
I do see many issues
a) This package is not a CPAN distro, it's a submodule of Test::Harness
Could you please point me to the part of Perl packaging guidelines that you believe to be violated by this?
b) This package doesn't install correctly - It wants to replace Test::Harness
How? I've double chcecked that this package don't have a single common provide with anything that is in RHEL5 with EPEL and Fedora 9. The SPEC file contains no Replaces nor Obsoletes tag. I also believe there is no file conflict. If you feel that there is a flaw in how I ensured to neither conflict with anything, nor replace anything please be more specific, e.g. provide a specific example.
c) This package will collide with a future upgrade of Test::Harness
There is a mechanism to deal with this -- I was asked to file a bug against F10 perl, so that included Test-Harness will be upgraded to TAP::Harness based version 3.10. I will do so shortly. There will be no need for this package to ever make it into F10 Rawhide then.
Only possible problem I can imagine would be a case where a hypothetical package would require perl-TAP-Harness instead of perl(TAP::Harness). In that case it is not a problem of this package, nor an unsolvable problem.
You are outsmarting yourselves.
Thanks for being polite and concentrating on providing well-verified facts.
DISAPPROVING.
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag|fedora-review- |fedora-review?, | |needinfo?(rc040203@freenet.d | |e)
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
------- Additional Comments From mmahut@redhat.com 2008-06-11 13:20 EST ------- Ralf?
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From tcallawa@redhat.com 2008-06-11 16:13 EST ------- FWIW, I disagree entirely with Ralf. Setting review back to +.
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs- |fedora-cvs?
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-06-12 12:05 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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |CLOSED Resolution| |NOTABUG Flag|needinfo?(rc040203@freenet.d| |e) |
------- Additional Comments From lkundrak@v3.sk 2008-06-12 13:34 EST ------- Thanks for assistance, Tom; Thanks for review, Marek; Thanks for CVS, Kevin; and thanks for constructive criticism, Ralf.
Imported and built. Followup in bug #451078.
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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|NOTABUG |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-TAP-Harness - Run Perl standard test scripts with statistics
https://bugzilla.redhat.com/show_bug.cgi?id=450553
------- Additional Comments From rc040203@freenet.de 2008-06-13 01:21 EST ------- (In reply to comment #9)
FWIW, I disagree entirely with Ralf. Setting review back to +.
OK, once more @RH's are abusing their positions.
package-review@lists.fedoraproject.org