[Bug 656084] Review Request: perl-CDB_File - Perl extension for access to cdb databases

bugzilla at redhat.com bugzilla at redhat.com
Sun Nov 28 00:55:12 UTC 2010


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

--- Comment #3 from Mark McKinstry <mmckinst at nexcess.net> 2010-11-27 19:55:11 EST ---
Petr,

Thank you for the review.

> FIX: BuildRequire perl(Exporter) as it can dual-live in the future
> (CDB_File.pm:8, http://search.cpan.org/~ferreira/Exporter/)
> FIX: BuildRequire perl(Test) as it can dual-live in the future (t/01main.t:2,
> http://search.cpan.org/~jesse/Test/)

Done.

Spec: http://mmckinst.nexcess.net/perl-CDB_File/perl-CDB_File.spec
SRPM:
http://mmckinst.nexcess.net/perl-CDB_File/perl-CDB_File-0.96-2.fc14.src.rpm
Scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=2630358

Can you explain what exactly dual-live means and how you figured out perl(Test) 
and perl(Exporter) needed to be BuildRequires?

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



More information about the package-review mailing list