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