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 #2 from Petr Pisar <[email protected]> 2012-01-06 03:21:13 EST ---
> 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.

> perl(File::Find),
Done.

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

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

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

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

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

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.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to