[Bug 758211] Review Request: perl-ExtUtils-MakeMaker - Create a module Makefile

[bugzilla at redhat.com]

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=758211

--- Comment #3 from Petr Šabata <psabata at redhat.com> 2012-01-06 03:38:40 EST ---
(In reply to comment #2)
> > TODO: The following BRs are not essential at the moment, however it might
> > change some time in the future.  Please, add them: perl(ExtUtils::Installed),
> Already presents on line 20.
> 

This was a typo, I meant ExtUtils::Install (in addition to Installed)

> > perl(File::Find),
> Done.

OK

> 
> > perl(File::Spec),
> Already presents on line 11.

True.


> 
> > perl(GetOpt::Long),
> Why? I cannot find any occurrence of this module in the sources.

./lib/ExtUtils/Command/MM.pm

> 
> > and perl(File::Temp).
> Done.

OK

> 
> > TODO: The following BRs are needed for the package to build in f17 mock/koji. 
> > Those were originally bundled:  perl(CPAN::Meta)
> Done

OK

> 
> > and perl(Parse::CPAN::Meta)
> Done.

OK

> 
> In addition, I've appended perl(ExtUtils::CBuilder) to BuildRequires to proceed
> optional t/xs.t test.
> 
> Updated spec and SRPM files are located on the same place.

-- 
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.