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 180532 - Review Request: perl-UNIVERSAL-can
Summary: Review Request: perl-UNIVERSAL-can
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-02-08 20:40 UTC by Jose Pedro Oliveira
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-02-18 16:39:33 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
build.log from mock (deleted)
2006-02-09 03:43 UTC, Jason Tibbitts
no flags Details

Description Jose Pedro Oliveira 2006-02-08 20:40:36 UTC
Spec Name or Url:
http://gsd.di.uminho.pt/jpo/software/fedora/perl-UNIVERSAL-can.spec

SRPM Name or Url:
http://gsd.di.uminho.pt/jpo/software/fedora/perl-UNIVERSAL-can-1.03-1.src.rpm

Description:
The UNIVERSAL class provides a few default methods so that all objects
can use them. Object orientation allows programmers to override these
methods in subclasses to provide more specific and appropriate behavior.

Some authors call methods in the UNIVERSAL class on potential invocants
as functions, bypassing any possible overriding. This is wrong and you
should not do it. Unfortunately, not everyone heeds this warning and
their bad code can break your good code.

Note:
This a new requirement of perl-Test-MockObject (which is tagged for an update)

Comment 1 Jason Tibbitts 2006-02-08 21:14:41 UTC
This package has BuildRequires: perl(Test::Simple) >= 0.60, which I can't
satisfy on FC4.  I suppose this is a perl 5.8.8 thing, which means that this
package will never build on FC4.

After commenting out the above BuildRequires: line, I find an unsatisfied
dependency on Perl(Test::Exception), which I can fulfill from extras.

I'll set up a mock build so I can test this with the current development release.



Comment 2 Jason Tibbitts 2006-02-09 03:42:38 UTC
Yep, a mock build with the current development tree dies due to the failed
Test::Exception dependency.  I'll attach the build log.

I hacked a bit to build an SRPM that with that additional dependency but it
still failed many of the tests.

I wonder if I'm doing something wrong.  How did you get this to build?

Comment 3 Jason Tibbitts 2006-02-09 03:43:57 UTC
Created attachment 124422 [details]
build.log from mock

Comment 4 Jose Pedro Oliveira 2006-02-10 10:22:58 UTC
Jason,

I messed up and tested the previous specfile in the wrong system. The previous
specfile didn't build correctely in devel (perl 5.8.8/Sup::Uplevel/Test::Warn
problems).

Right now there are two new versions of UNIVERSAL::can in CPAN and at least the
latest one passes all tests in devel (according to the Changelog it now includes
workarounds to the weird problems in Test::Warn).  I believe the problems are
due to Sub::Uplevel in perl 5.8.8 (the test suite generates a lot of noise) but
I only have time to look into it next week.

Diff from UNIVERSAL-can-1.03 to UNIVERSAL-can-1.11
http://search.cpan.org/diff?from=UNIVERSAL-can-1.03&to=UNIVERSAL-can-1.11

New SRPM:
http://gsd.di.uminho.pt/jpo/software/fedora/perl-UNIVERSAL-can-1.11-1.src.rpm

The specfile link is the same.

jpo


Comment 5 Jason Tibbitts 2006-02-12 23:13:31 UTC
This package builds cleanly in mock, but on the devel branch only due to the
Test::Simple >= 0.60 requirement.  As far as I can tell there is no requirement
that packages build on the release branch so this is not a blocker.  However, I
just commented out the Test::Simple requirement and the package built fine and
passes all tests on FC4 and FC3.

rpmlint is silent.
The package meets the naming and packaging guidelines.
The specfile is properly named and follows the recomment Perl template.
The source file matches upstream and provided signatures verify with cpansign -v.
Licence: tag is proper and matches the package.
BuildRequires: is proper.

Approved, but you might want to investigate dropping the Test::Simple
requirement altogether and building the package on the release branch.

Comment 6 Jose Pedro Oliveira 2006-02-18 16:39:33 UTC
Built for devel only.

I am also targeting the perl-test-MockObject to devel only.



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