Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 189080 - Review Request: perl-Module-Info
Summary: Review Request: perl-Module-Info
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jose Pedro Oliveira
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 189081
TreeView+ depends on / blocked
 
Reported: 2006-04-15 18:03 UTC by Steven Pritchard
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-07 20:46:05 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Steven Pritchard 2006-04-15 18:03:46 UTC
Spec URL: http://ftp.kspei.com/pub/steve/rpms/perl-Module-Info/perl-Module-Info.spec
SRPM URL: http://ftp.kspei.com/pub/steve/rpms/perl-Module-Info-0.290-1.src.rpm
Description:
Module::Info gives you information about Perl modules without actually
loading the module. It actually isn't specific to modules and should work
on any perl code.

Comment 1 Jose Pedro Oliveira 2006-04-17 22:01:07 UTC
Steven,

Before starting the review could you make the following changes:
 * update to version 0.30
 * replace perl by %{__perl} (better consistency)
 * drop the line "find $RPM_BUILD_ROOT -type d ..." as it it not needed for
   noarch perl packages

tia,
jpo

Comment 3 Jose Pedro Oliveira 2006-04-22 19:05:07 UTC
NEEDSWORK

MD5SUMS:
c7dc1e91123ea940d13eaa0cce396ffc  perl-Module-Info-0.30-1.src.rpm

a1e4e39cceda93dc0ab2d296b5075d76  Module-Info-0.30.tar.gz
dab0c42046cd186526f35a1f951758fd  perl-Module-Info.spec


Good:
* CPAN tarball MD5 digest matched the inclued tarball
* URL and Source URL are valid
* License verified (main POD page, Build.PL)
* perl(:MODULE_COMPAT_xxx) present
* perl vendor libs present
* File permissions are ok
* No critical bugs in http://rt.cpan.org/NoAuth/Bugs.html?Dist=Module-Info

Needswork:
* missing build requirements
  BuildRequires:  perl(version)
  BuildRequires:  perl(Test::Pod) 

* missing requirement
  Requires:       perl(version)


Comment 4 Jason Tibbitts 2006-04-22 23:04:16 UTC
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?

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?

Just trying to understand these things so I can do better reviews.

Comment 5 Jose Pedro Oliveira 2006-04-22 23:17:21 UTC
(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

Comment 6 Steven Pritchard 2006-04-22 23:22:54 UTC
When I tried BR: perl(version), "make test" still complained about missing
version.pm, so I dropped the line.

I'm guessing that's an upstream bug?

Comment 7 Steven Pritchard 2006-04-22 23:25:09 UTC
[...]
t/n3_version.............ok                                                  
        6/14 skipped: version.pm found, can not test
[...]
$ rpm -q perl-version
perl-version-0.59-1.fc4


Comment 8 Jose Pedro Oliveira 2006-04-22 23:28:20 UTC
No. It found the version module (... version.pm found ...).

The n3_version test script has two subset tests:
  one subset if it finds the version object,
  a second if it doesn't found,
  and it skips one of them. 

jpo

Comment 9 Jason Tibbitts 2006-04-23 02:44:09 UTC
Sorry for being dense; I understand now what you were asking for.

Comment 10 Jose Pedro Oliveira 2006-04-23 13:58:13 UTC
(In reply to comment #9)
> Sorry for being dense; I understand now what you were asking for.

No problem Jason.

I am starting to collect a few cases where requirements aren't detected by the
perl.req script. Will send you a draft version in a couple of minutes. 

jpo

Comment 11 Jose Pedro Oliveira 2006-05-04 16:34:33 UTC
Steven,

Please see comment 3.

jpo

Comment 12 Steven Pritchard 2006-05-04 16:44:47 UTC
Thanks.  I should be getting back to this over the weekend at the latest.

Comment 13 Steven Pritchard 2006-05-06 18:17:15 UTC
OK, so I'm an idiot, I read "version.pm found, can not test" wrong.  Sorry about
that...

I've made the requested changes in -2.

 http://ftp.kspei.com/pub/steve/rpms/perl-Module-Info-0.30-2.src.rpm

Comment 14 Jose Pedro Oliveira 2006-05-06 18:34:26 UTC
No problem.
 
MD5SUMS:
cba0296eedd5f24439aad1716d4caf76  perl-Module-Info-0.30-2.src.rpm

a1e4e39cceda93dc0ab2d296b5075d76  Module-Info-0.30.tar.gz
66ce44d44e3e2c1b7c645aa55f89f6a3  perl-Module-Info.spec

APPROVED.

Comment 15 Steven Pritchard 2006-05-07 20:46:05 UTC
Imported into CVS, branches created, and builds done.


Note You need to log in before you can comment on or make changes to this bug.