Skip Menu |
 

This queue is for tickets about the HTTP-Proxy CPAN distribution.

Report information
The Basics
Id: 56259
Status: resolved
Priority: 0/
Queue: HTTP-Proxy

People
Owner: Nobody in particular
Requestors: KENTNL [...] cpan.org
SREZIC [...] cpan.org
Cc: CARNIL [...] cpan.org
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.24
Fixed in: 0.302



Subject: Pod coverage test is failing with perl 5.12.0 RC0
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 100
Download (untitled) / with headers
text/plain 100b
See http://www.cpantesters.org/cpan/report/7033492 for a sample tester report. Regards, Slaven
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.13-32706-1373953817-1761.56259-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 334
Download (untitled) / with headers
text/plain 334b
Despite being marked as fixed in 0.300, this ticket is *not* fixed in 0.300, tests still fail for non-threaded perls. Sample fail report: http://www.cpantesters.org/cpan/report/caf9e7ba-c9c7-11e2-a19f-6eea979cd64d It would probably be a good idea to make pod tests author tests, they make user installs fail unnecessarily. Thanks,
From philippe.bruhat [...] free.fr Tue Jul 16 05: 15:14 2013
MIME-Version: 1.0
X-Spam-Status: No, score=-5.014 tagged_above=-99.9 required=10 tests=[AWL=1.885, BAYES_00=-1.9, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_HI=-5] autolearn=ham
In-Reply-To: <rt-4.0.13-32706-1373953817-1919.56259-5-0 [...] rt.cpan.org>
Content-Disposition: inline
X-Spam-Flag: NO
X-RT-Interface: API
References: <RT-Ticket-56259 [...] rt.cpan.org> <rt-4.0.13-32706-1373953817-1919.56259-5-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <20130716090632.GC7525 [...] zlott>
content-type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -5.014
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 62D8C240DAD for <cpan-bug+HTTP-Proxy [...] hipster.bestpractical.com>; Tue, 16 Jul 2013 05:07:03 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id mfTLWrSPEGMQ for <cpan-bug+HTTP-Proxy [...] hipster.bestpractical.com>; Tue, 16 Jul 2013 05:07:01 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 772EE240F47 for <bug-HTTP-Proxy [...] rt.cpan.org>; Tue, 16 Jul 2013 05:07:01 -0400 (EDT)
Received: (qmail 27450 invoked by alias); 16 Jul 2013 09:07:00 -0000
Received: from zlonk.bruhat.net (HELO ks354402.kimsufi.com) (91.121.102.217) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Tue, 16 Jul 2013 02:06:54 -0700
Received: from localhost.localdomain ([127.0.0.1] helo=zlott) by ks354402.kimsufi.com with esmtp (Exim 4.80) (envelope-from <philippe.bruhat [...] free.fr>) id 1Uz1Dd-0003t8-OL for bug-HTTP-Proxy [...] rt.cpan.org; Tue, 16 Jul 2013 11:06:49 +0200
Received: from book by zlott with local (Exim 4.80) (envelope-from <philippe.bruhat [...] free.fr>) id 1Uz1DM-0004WK-LY for bug-HTTP-Proxy [...] rt.cpan.org; Tue, 16 Jul 2013 11:06:32 +0200
Delivered-To: cpan-bug+HTTP-Proxy [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #56259] Pod coverage test is failing with perl 5.12.0 RC0
User-Agent: Mutt/1.5.21 (2010-09-15)
Return-Path: <philippe.bruhat [...] free.fr>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: cpan-bug+HTTP-Proxy [...] hipster.bestpractical.com
X-RT-Mail-Extension: http-proxy
Date: Tue, 16 Jul 2013 11:06:32 +0200
X-Spam-Level:
To: Andreas Koenig via RT <bug-HTTP-Proxy [...] rt.cpan.org>
From: "Philippe Bruhat (BooK)" <philippe.bruhat [...] free.fr>
RT-Message-ID: <rt-4.0.13-11583-1373966114-1834.56259-0-0 [...] rt.cpan.org>
Content-Length: 768
Download (untitled) / with headers
text/plain 768b
On Tue, Jul 16, 2013 at 01:50:17AM -0400, Andreas Koenig via RT wrote: Show quoted text
> Queue: HTTP-Proxy > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=56259 > > > Despite being marked as fixed in 0.300, this ticket is *not* fixed in 0.300, tests still fail for non-threaded perls. Sample fail report: > > http://www.cpantesters.org/cpan/report/caf9e7ba-c9c7-11e2-a19f-6eea979cd64d > > It would probably be a good idea to make pod tests author tests, they make user installs fail unnecessarily. >
Isn't the issue the fact that the test fails when run with an unthreaded Perl? -- Philippe Bruhat (BooK) Putting beauty before brains is the surest way to wind up with neither. (Moral from Groo The Wanderer #24 (Epic))
MIME-Version: 1.0
In-Reply-To: <rt-4.0.13-11583-1373966114-1834.56259-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <RT-Ticket-56259 [...] rt.cpan.org> <rt-4.0.13-32706-1373953817-1919.56259-5-0 [...] rt.cpan.org> <20130716090632.GC7525 [...] zlott> <rt-4.0.13-11583-1373966114-1834.56259-0-0 [...] rt.cpan.org>
Content-Type: text/html; charset="utf-8"
Message-ID: <rt-4.0.18-3211-1403119654-937.56259-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 2453

On 2013-07-16 21:15:14, Philippe.Bruhat@free.fr wrote:
> Isn't the issue the fact that the test fails when run with an
> unthreaded Perl?

Yes... and no. The only failure is 02pod-coverage.t

Yes, the test fails doing Pod::Coverage for HTTP::Proxy::Engine::Threaded

However, thats immaterial: Because pod coverage tests really shouldn't be run on end user machines, because they cause problems for non-reasons that require user intervention to step around, which is unncessecary.

Here, Pod::Coverage is attempting to load the one module in the dist that requires threads, because its not smart enough not to, and that leads to the error.

 

( Even on the *best* of days, a Pod::Coverage test failing can be of no use to a user. "OH NOES DEVELOPER FORGOT TO DOCUMENT THIS METHOD, BETTER NOT INSTALL!  " ;) )


perl -d:Confess -T -Ilib t/02pod-coverage.t
1..18
ok 1 - Pod coverage on HTTP::Proxy
ok 2 - Pod coverage on HTTP::Proxy::HeaderFilter
ok 3 - Pod coverage on HTTP::Proxy::Engine
ok 4 - Pod coverage on HTTP::Proxy::FilterStack
ok 5 - Pod coverage on HTTP::Proxy::BodyFilter
ok 6 - Pod coverage on HTTP::Proxy::BodyFilter::tags
ok 7 - Pod coverage on HTTP::Proxy::BodyFilter::simple
ok 8 - Pod coverage on HTTP::Proxy::BodyFilter::lines
ok 9 - Pod coverage on HTTP::Proxy::BodyFilter::htmltext
ok 10 - Pod coverage on HTTP::Proxy::BodyFilter::htmlparser
ok 11 - Pod coverage on HTTP::Proxy::BodyFilter::complete
ok 12 - Pod coverage on HTTP::Proxy::BodyFilter::save
ok 13 - Pod coverage on HTTP::Proxy::Engine::ScoreBoard
ok 14 - Pod coverage on HTTP::Proxy::Engine::Legacy
ok 15 - Pod coverage on HTTP::Proxy::Engine::NoFork
not ok 16 - Pod coverage on HTTP::Proxy::Engine::Threaded
#   Failed test 'Pod coverage on HTTP::Proxy::Engine::Threaded'
#   at /home/kent/perl5/perlbrew/perls/perl-5.20.0/lib/site_perl/5.20.0/Test/Pod/Coverage.pm line 126.
# HTTP::Proxy::Engine::Threaded: requiring 'HTTP::Proxy::Engine::Threaded' failed
ok 17 - Pod coverage on HTTP::Proxy::HeaderFilter::simple
ok 18 - Pod coverage on HTTP::Proxy::HeaderFilter::standard
# Looks like you failed 1 test of 18.
 

MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-3211-1403119654-937.56259-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <RT-Ticket-56259 [...] rt.cpan.org> <rt-4.0.13-32706-1373953817-1919.56259-5-0 [...] rt.cpan.org> <20130716090632.GC7525 [...] zlott> <rt-4.0.13-11583-1373966114-1834.56259-0-0 [...] rt.cpan.org> <rt-4.0.18-3211-1403119654-937.56259-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-8869-1429344592-1258.56259-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 407
Download (untitled) / with headers
text/plain 407b
On Wed Jun 18 15:27:34 2014, KENTNL wrote: Show quoted text
> On 2013-07-16 21:15:14, Philippe.Bruhat@free.fr wrote:
> > Isn't the issue the fact that the test fails when run with an > > unthreaded Perl?
> > ( Even on the *best* of days, a Pod::Coverage test failing can be of > no use to > a user. "OH NOES DEVELOPER FORGOT TO DOCUMENT THIS METHOD, BETTER NOT > INSTALL! > " ;) ) >
This was fixed by moving to dzil. :-)


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.