Bug 180532

Summary: Review Request: perl-UNIVERSAL-can
Product: [Fedora] Fedora Reporter: Jose Pedro Oliveira <jose.p.oliveira.oss>
Component: Package ReviewAssignee: Jason Tibbitts <j>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-02-18 16:39:33 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 163779    
Attachments:
Description Flags
build.log from mock none

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.