Skip Menu |
 

This queue is for tickets about the threads CPAN distribution.

Report information
The Basics
Id: 48570
Status: resolved
Priority: 0/
Queue: threads

People
Owner: Nobody in particular
Requestors: adamk [...] cpan.org
Cc:
AdminCc:

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



Subject: Incorrectly skips tests (UNKNOWN) when it should do an NA
Download (untitled) / with headers
text/plain 534b
The threads module is invalid if the Perl it is installed into does not have thread support. And yet, it will still build and test even when not on a threading Perl. This results in a large number of false-UNKNOWN (which basically says "Testing was unreliable for this, or crashed") results for this module. What it probably should do instead is to check for thread support during the Makefile.PL and return NA if the Perl does not support threads. (This is usually done by returning success exit(0) without generating a Makefile)
RT-Send-CC: perl5-porters [...] perl.org
On Mon Aug 10 01:14:50 2009, ADAMK wrote: Show quoted text
> The threads module is invalid if the Perl it is installed into does not > have thread support. > > And yet, it will still build and test even when not on a threading Perl. > > This results in a large number of false-UNKNOWN (which basically says > "Testing was unreliable for this, or crashed") results for this module. > > What it probably should do instead is to check for thread support during > the Makefile.PL and return NA if the Perl does not support threads. > > (This is usually done by returning success exit(0) without generating a > Makefile)
The original philosophy for this module is that 'threads' and its ilk are allowed to be installed in an unthreaded Perl. threads.pm checks if the Perl is threaded, and 'die's if it is not. This allows apps to be written such that they can be supported on both threaded and unthreaded Perls using a construct similar to this: my $i_am_threaded; BEGIN { $i_am_threaded = eval 'use threads; 1'; } Backward compatibility concerns also dictate that this behavior not be modified.
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 699b
On Mon Aug 10 01:14:50 2009, ADAMK wrote: Show quoted text
> The threads module is invalid if the Perl it is installed into does not > have thread support. > > And yet, it will still build and test even when not on a threading Perl. > > This results in a large number of false-UNKNOWN (which basically says > "Testing was unreliable for this, or crashed") results for this module.
Of further consideration, I believe it would be appropriate for me to add at least one test that will pass on a non-threaded Perl. This will allow the module to be installed on non-threaded Perls (which is required for backward compatibility), and to 'pass' at the same time. This should eliminate those ambiguous UNKNOWN results.
CC: adamk [...] cpan.org
Subject: Re: [rt.cpan.org #48570] Incorrectly skips tests (UNKNOWN) when it should do an NA
Date: Tue, 11 Aug 2009 05:00:41 +1000
To: bug-threads [...] rt.cpan.org
From: Adam Kennedy <adamkennedybackup [...] gmail.com>
Removing spurious UNKNOWN results via a single passing test would be the next best way to fix this, yes. The Padre "Percentage likely to install" dependency graph appreciates this. :) Adam K 2009/8/11 Jerry D. Hedden via RT <bug-threads@rt.cpan.org> Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=48570 > > > On Mon Aug 10 01:14:50 2009, ADAMK wrote:
> > The threads module is invalid if the Perl it is installed into does not > > have thread support. > > > > And yet, it will still build and test even when not on a threading Perl. > > > > This results in a large number of false-UNKNOWN (which basically says > > "Testing was unreliable for this, or crashed") results for this module.
> > Of further consideration, I believe it would be appropriate for me to add > at least one test that will pass on a non-threaded Perl. This will > allow the module to be installed on non-threaded Perls (which is > required for backward compatibility), and to 'pass' at the same time. > This should eliminate those ambiguous UNKNOWN results. >


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.