Skip Menu |
 

This queue is for tickets about the List-MoreUtils CPAN distribution.

Report information
The Basics
Id: 82123
Status: resolved
Priority: 0/
Queue: List-MoreUtils

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

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



Subject: Re: [rt.cpan.org #82123] AutoReply: Useless use of a variable in void context
Date: Fri, 21 Dec 2012 18:47:22 -0800
To: Bugs in List-MoreUtils via RT <bug-List-MoreUtils [...] rt.cpan.org>
From: Karen Etheridge <ether [...] cpan.org>
Download (untitled) / with headers
text/plain 276b
Show quoted text
> I've received several cpantesters failure reports that seem to be based > in issues in List::MoreUtils:
Two more today: http://www.cpantesters.org/cpan/report/c46652cc-4ac3-11e2-a0e7-605e6fb4b169 http://www.cpantesters.org/cpan/report/f8fe95a4-4a90-11e2-8e29-0acdbf2719ed
Is this still an active issue?
Download (untitled) / with headers
text/plain 252b
On 2015-03-09 07:48:11, REHSACK wrote: Show quoted text
> Is this still an active issue?
I haven't seen these reports in a while. Is there any guess what the problem might have been? If we can figure it out, we can confirm that it's no longer an issue in the new code.
Download (untitled) / with headers
text/plain 1.5k
11:39 <@haarg> ether: https://metacpan.org/source/RURBAN/List-MoreUtils-0.33_005/lib/List/MoreUtils.pm#L362 11:42 <@haarg> ether: that line of code is buggy and will warn under -w 11:42 <@ether> haarg: can you clarify? how is -w coming into play? 11:43 <@haarg> make test 11:43 <@ether> in the harness itself? I don't see it in the arg string 11:43 <@haarg> try again in 2012 11:44 <@haarg> make test used to apply -w to all test scripts 11:45 <@[Sno]> haarg: maybe that's at https://github.com/perl5-utils/List-MoreUtils/blob/master/lib/List/MoreUtils/PP.pm#L335 now 11:46 <@haarg> basically, it's an old problem in an old dev release, combined with an old EUMM, and an older test setup ether was using. 11:46 <@[Sno]> I'm just walking over tickets and see where I have questions to fix open issues (more important than wish-list items) 11:46 <@ether> s/ether/the smoker/ 11:47 <@haarg> i mean that you were using Test::Warnings without an env check 11:47 <@haarg> iirc 11:47 <@haarg> doesn't really matter though. tracked down the problem, and it isn't a problem. 11:48 <@ether> Test::Warnings turned the warnings fatal, but we would have seen those warnings anyway in the test output because of -w and the lack of "no warnings 'void'" 11:48 <@ether> I no longer enable Test::Warnings for smokers, at least -- learned that lesson :) 11:48 <@ether> nor use warnings FATAL => 'all' 11:50 <@haarg> ether: it would have shown up, but only if you had installed the dev release of List::MoreUtils 11:50 <@haarg> unlikely to be widely noticed So I think we can mark this resolved.
Thanks for the summary, ether. I close it as resolved for now.


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.