[Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

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

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #4 from Petr Šabata <psabata@xxxxxxxxxx> ---
Issues:

Missing build-time dependencies:
perl(Exporter)
perl(Test)

Lines 30 and 31 are useless.  None of the files has executable bits set.
Line 32 is wrong.  Modules aren't scripts and aren't supposed to contain
shebangs.
Line 43 may be removed.  Done by rpmbuild.

Preferrably substitute command macros with simple calls.

The license is not mentioned anywhere in the upstream tarball, not even on the
CPAN module webpage.  Please, ask upstream for clarification.

-- 
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=8yoqHU33R9&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]