Skip Menu |

This queue is for tickets about the Smart-Comments CPAN distribution.

Report information
The Basics
Id: 14655
Status: resolved
Priority: 0/
Queue: Smart-Comments

People
Owner: Nobody in particular
Requestors: mcogan [...] jupitermedia.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.0.1
Fixed in: (no value)



Subject: Smrt::Comments doesn't pass it's own tests under perl 5.6

Message body is not shown because it is too large.

Date: Tue, 20 Sep 2005 14:09:28 +1000
To: bug-Smart-Comments [...] rt.cpan.org
Subject: Re: [cpan #14655] Smrt::Comments doesn't pass it's own tests under perl 5.6
From: Damian Conway <thoughtstream [...] gmail.com>
RT-Send-Cc:
Guest via RT wrote: Show quoted text
> This message about Smart-Comments was sent to you by guest <> via rt.cpan.org > > Full context and any attached attachments can be found at: > <URL: https://rt.cpan.org/Ticket/Display.html?id=14655 > > > SmartCommnetrs 1.01 > This is perl, v5.6.1 built for i386-linux > Linux romeo 2.4.27 #2 SMP Tue Sep 14 22:39:30 MST 2004 i686 unknown > > The test suite fails dramatically under perl 5.6.1 (dump follows). It appears that opening a filehandle to "print" to a scalar reference isn't supported in this version of perl; the test suitres rely on this to capture ouput to STDOUT (see, eg, t/assert.t line 6). Consequently, the test suites create lots of files named 'SCALAR(...)', and fail because the strings they are testing are all empty. > > The module itself seems to work fine, although without a usable test suite under 5.6.1, I can't tell for sure.
Hmmmm. I'll have to look at some other mechanism for doing those tests. Thanks for the report! Damian
I have attached a fixed version of the test suite. Seems to work for 5.8 and 5.6, and was tested under v5.8.5 built for x86_64-linux-thread-multi and v5.6.1 built for sun4-solaris I used an additional module, IO::Scalar to capture STDERR which will need to be added to prereqs. Thanks, David M. Bradford
Download new_tests.tar.gz
application/x-gzip-compressed 2.6k

Message body not shown because it is not plain text.

Smart::Comments uses features of perl 5.8, so now Build.PL requires it. -- Alexandr Ciornii, http://chorny.net