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