Skip Menu |

This queue is for tickets about the Test-Portability-Files CPAN distribution.

Report information
The Basics
Id: 21631
Status: resolved
Priority: 0/
Queue: Test-Portability-Files

People
Owner: abraxxa [...] cpan.org
Requestors: chris+rt [...] chrisdolan.net
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.05
Fixed in: 0.06



Subject: [PATCH] .options() cannot take a false value
I'm trying to turn off Amiga/Mac filename length checking via options(test_amiga_length => 0, test_mac_length => 0); However, the options() sub checks for truth of the value, not existence. So, this fails. The attached simple patch fixes this problem. -- Chris
Subject: negative_options.patch
--- /Users/chris/perl/lib/perl5/Test/Portability/Files.pm 2006-03-09 21:07:51.000000000 -0600 +++ /tmp/Files.pm 2006-09-22 23:05:12.000000000 -0500 @@ -276,12 +276,13 @@ sub options { my %opts = @_; for my $test (keys %tests) { - $tests{$test} = $opts{"test_$test"} if $opts{"test_$test"} + $tests{$test} = $opts{"test_$test"} if exists $opts{"test_$test"} } for my $opt (keys %options) { - $options{$opt} = $opts{$opt} if $opts{$opt} + $options{$opt} = $opts{$opt} exists if $opts{$opt} } - @tests{keys %tests} = (1)x(keys %tests) if $opts{all_tests}; + @tests{keys %tests} = ($opts{all_tests})x(keys %tests) + if exists $opts{all_tests}; } =item C<test_name_portability()>
From: frequency [...] cpan.org
On Sat Sep 23 00:07:03 2006, CDOLAN wrote: Show quoted text
> I'm trying to turn off Amiga/Mac filename length checking via > > options(test_amiga_length => 0, test_mac_length => 0); > > However, the options() sub checks for truth of the value, not existence. > So, this fails. The attached simple patch fixes this problem. > > -- Chris >
Hi: I've applied this patch to Debian, so it should work properly for libtest-portability-files-perl there. However, I'm adding this reply mainly to ping the maintainer to apply this patch upstream. Cheers, Jonathan
subscribing to get notified when fixed.
From: blue [...] thisisnotmyrealemail.com
Wanting this 6-yearl old patch also applied. Is this module even being maintained anymore?
Am Mo 26. Mär 2012, 07:13:12, blue schrieb: Show quoted text
> Wanting this 6-yearl old patch also applied. Is this module even being > maintained anymore?
I'm trying to get the maintainership for this module since last year to fix some of the bugs.
fixed in 0.06