[Bug 192575] Review Request: perl-Pipeline

bugzilla at redhat.com bugzilla at redhat.com
Thu May 25 18:53:56 UTC 2006


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-Pipeline


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=192575





------- Additional Comments From steve at silug.org  2006-05-25 14:46 EST -------
(In reply to comment #1)
> Some notes from the test suite:
> 
> t/11async................Use of uninitialized value in numeric eq (==) at
> /builddir/build/BUILD/Pipeline-3.12/blib/lib/Pipeline/Store.pm line 33 during
> global destruction.
> Unbalanced string table refcount: (1) for "debug" during global destruction.
> Unbalanced string table refcount: (1) for "dispatcher" during global destruction.
> Unbalanced string table refcount: (1) for "parent" during global destruction.
> Unbalanced string table refcount: (1) for "store" during global destruction.
> Scalars leaked: 1
> ok
> 
> Not sure what's up here.

I opened a ticket for this.  http://rt.cpan.org/Ticket/Display.html?id=19481

> t/98compile..............skipped
>         all skipped: - do not have File::Find::Rule installed
> t/99pod..................skipped
>         all skipped: - do not have Find::File::Rule or Pod::Checker installed
[...]
> Acme::Colour isn't in extras, but the others are and adding them as BR:s gives
> better coverage:

Pod::Checker is a core module, so I just needed to add File::Find::Rule.

Fixed in -2, in CVS.

> t/03colour...............skipped
>         all skipped: - do not have Acme::Colour installed
[...]
> I don't think there's any need to package Acme::Colour just for this; it's kind
> of, uhh, limited in scope anyway.

Agreed.  Anything in the Acme namespace is probably of dubious value.

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




More information about the Fedora-package-review mailing list