[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