[Bug 752838] Review Request: perl-TAP-Formatter-JUnit - Harness output delegate for JUnit output

bugzilla at redhat.com bugzilla at redhat.com
Wed Nov 16 06:02:55 UTC 2011


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=752838

Parag AN(पराग) <panemade at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |panemade at gmail.com
         AssignedTo|nobody at fedoraproject.org    |panemade at gmail.com
               Flag|                            |fedora-review+

--- Comment #1 from Parag AN(पराग) <panemade at gmail.com> 2011-11-16 01:02:54 EST ---
Review:-

+ koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3518023
+ rpmlint on package gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.
+ source match with upstream as (sha1sum)
3ba936489fff2f9c305f5d907595fb7e6d024c7a 
../SOURCES/TAP-Formatter-JUnit-0.08.tar.gz
3ba936489fff2f9c305f5d907595fb7e6d024c7a  TAP-Formatter-JUnit-0.08.tar.gz

+ make test gave
All tests successful.
Files=8, Tests=68, 28 wallclock secs ( 0.05 usr  0.01 sys +  3.93 cusr  0.70
csys =  4.69 CPU)
+ Package: perl-TAP-Formatter-JUnit-0.08-1.fc17.noarch
Provides: perl(TAP::Formatter::JUnit) = 0.08
perl(TAP::Formatter::JUnit::Session)
Requires: /usr/bin/perl perl(Class::Field) perl(File::Path) perl(File::Slurp)
perl(Getopt::Long) perl(IO::File) perl(Pod::Usage) perl(Storable)
perl(TAP::Formatter::Console) perl(TAP::Formatter::Console::Session)
perl(TAP::Formatter::JUnit) perl(TAP::Formatter::JUnit::Session)
perl(TAP::Parser) perl(TAP::Parser::Aggregator) perl(XML::Generator) perl(base)
perl(strict) perl(warnings)

+ Follows packaging guidelines

Suggestions:
1) For the active fedora releases you can remove following from spec
 a) buildroot
 b) cleaning of buildroot in %install
 c) %clean section
 d) %defattr(-,root,root,-)

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


More information about the package-review mailing list