Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Module-Info
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=189080
------- Additional Comments From jpo(a)di.uminho.pt 2006-04-22 19:17 EST -------
(In reply to comment #4)
I'm a bit confused about your needswork items:
Last week, BuildRequires: perl wasn't permitted; now it's optional, but it seems
that here you're requiring it. Is that what you're asking for, or am I confused?
No, I am not requiring perl. Module::Info can be extened to use version
objects provided by the perl version module.
$ rpm -q --whatprovides "perl(version)"
perl-version-0.59-1.fc5
And this package already (as I look at it now, at least) has the
usual
Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
so why would a separate Requires: perl(version) be needed?
perl(version) represents a perl module available in CPAN (and already in Extras).
the statement "%{__perl} -V:version" prints one of the perl configuration
variables.
$ perl -V:version
version='5.8.8';
Note: the rpm perl scripts fail to detect required modules. They only look for
use statements and ignore require statements (among other things they fail to
look for like system calls, backticks, open with pipes, ...).
Hope this helps,
jpo
--
Configure bugmail:
https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.