[Bug 189080] Review Request: perl-Module-Info

[bugzilla at redhat.com]

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