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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |panemade@xxxxxxxxx
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |panemade@xxxxxxxxx
               Flag|                            |fedora-review+

--- Comment #1 from Parag AN(पराग) <panemade@xxxxxxxxx> 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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]