[Bug 1139041] Review Request: perl-Devel-Timer - Track and report execution time for parts of code
bugzilla at redhat.com
bugzilla at redhat.com
Tue Oct 7 11:11:09 UTC 2014
https://bugzilla.redhat.com/show_bug.cgi?id=1139041
--- Comment #1 from Petr Ĺ abata <psabata at redhat.com> ---
So, for the build time deps... Your Makefile.PL uses the bundled
inc::Module::Install. That means you need to list the dependencies of the
bundled distribution, too. In your case that means the following:
- Config, inc/Module/Install/Can.pm:5
- Cwd, inc/Module/Install.pm:22
- Fcntl, inc/Module/Install/Makefile.pm:7
- File::Find, inc/Module/Install.pm:23
- inc/Module/Install.pm:24
- ExtUtils::MM_Unix, inc/Module/Install/Metadata.pm:323,
inc/Module/Install/Metadata.pm:332
Other missing build time dep, used by the testsuite:
- lib, t/my-timer.t:8
You also list a dependency on Carp which is not used or even mentioned anywhere
in the tarball. Please, remove it.
Nitpick: Perhaps you could change the el5 check in the %install section to be
the same as the check around the BuildRoot tag? Or the other way around. Just
to be consistent.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
More information about the package-review
mailing list