[Bug 979676] Review Request: perl-Test-TrailingSpace - Test for trailing space in source files

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=979676

Parag AN(पराग) <panemade@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |panemade@xxxxxxxxx
              Flags|needinfo?(panemade@xxxxxxxx |fedora-review+
                   |m)                          |

--- Comment #3 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
Still I am getting in local mock build
Error: No Package found for perl(File::Find::Object::Rule)

so, waiting for this package. But, anyway I can review this using koji scratch
build.

Review:
+ koji scratch build ->
http://koji.fedoraproject.org/koji/taskinfo?taskID=5607384

+ rpmlint on rpms gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.


+ license is "MIT" and included in source file.

+ Source verified with upstream as
srpm tarball : e81787f1fad34093d8e19f26ae8a5cfcf658ff577ba010a40a586fa4237f9ed2
upstream tarball :
e81787f1fad34093d8e19f26ae8a5cfcf658ff577ba010a40a586fa4237f9ed2

+ make test gaveAll tests successful.
Files=4, Tests=8,  0 wallclock secs ( 0.03 usr  0.00 sys +  0.22 cusr  0.04
csys =  0.29 CPU)

+ Package perl-Test-TrailingSpace-0.0203-1.fc20
Provides: perl(Test::TrailingSpace) = 0.0203 perl-Test-TrailingSpace =
0.0203-1.fc20
Requires: perl(File::Find::Object::Rule) >= 0.0301 perl(Test::More)
perl(autodie) perl(strict) perl(warnings)

Suggestions:
1) remove following requires. let rpm find and install them for this package.
Requires:       perl(autodie)
Requires:       perl(File::Find::Object::Rule)
Requires:       perl(Test::More)

APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=6LuxlQhEmR&a=cc_unsubscribe
_______________________________________________
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]