Skip Menu |

This queue is for tickets about the Pod-Inherit CPAN distribution.

Report information
The Basics
Id: 106931
Status: resolved
Priority: 0/
Queue: Pod-Inherit
People
Owner: Nobody in particular
Requestors: ribasushi [...] leporine.io
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Sep 07 09:25:08 2015 ribasushi [...] leporine.io - Ticket created
Subject: Tests fail due to newly introduced warnings in File::Path
The change https://metacpan.org/source/RICHE/File-Path-2.11/Changes#L4 trickles through Path::Class into t/06dir.t of this module, with the following result: t/06dir.t ......... 1/? # Failed test 'no warnings' # at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/site_perl/5.8.5/Test/NoWarnings.pm line 45. # There were 1 warning(s) # Previous test 1 'use Pod::Inherit;' # t/output/files for t/output/files: No such file or directory at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/site_perl/5.8.5/Path/Class/Dir.pm line 155. # at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/5.8.5/File/Path.pm line 65. # File::Path::_error(HASH(0x1f0f7e0), "t/output/files", "t/output/files") called at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/5.8.5/File/Path.pm line 346 # File::Path::_rmtree(HASH(0x1f0f7e0), ARRAY(0x1aa5200)) called at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/5.8.5/File/Path.pm line 320 # File::Path::rmtree("t/output/files") called at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/site_perl/5.8.5/Path/Class/Dir.pm line 155 # Path::Class::Dir::rmtree(Path::Class::Dir=HASH(0x1bbf140)) called at t/06dir.t line 16 # # Looks like you failed 1 test of 33. t/06dir.t ......... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/33 subtests
  Mon Sep 07 13:19:12 2015 SREZIC [...] cpan.org - Correspondence added
On 2015-09-07 09:25:08, RIBASUSHI wrote: Show quoted text
> The change https://metacpan.org/source/RICHE/File-Path-2.11/Changes#L4 > trickles through Path::Class into t/06dir.t of this module, with the > following result: > > t/06dir.t ......... 1/? > # Failed test 'no warnings' > # at > /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/site_perl/5.8.5/Test/NoWarnings.pm > line 45. > # There were 1 warning(s) > # Previous test 1 'use Pod::Inherit;' > # t/output/files for t/output/files: No such file or directory > at > /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/site_perl/5.8.5/Path/Class/Dir.pm > line 155. > # at > /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/5.8.5/File/Path.pm > line 65. > # File::Path::_error(HASH(0x1f0f7e0), "t/output/files", > "t/output/files") called at > /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/5.8.5/File/Path.pm > line 346 > # File::Path::_rmtree(HASH(0x1f0f7e0), ARRAY(0x1aa5200)) called > at /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/5.8.5/File/Path.pm > line 320 > # File::Path::rmtree("t/output/files") called at > /home/rabbit/perl5/perlbrew/perls/5.8.5_trv/lib/site_perl/5.8.5/Path/Class/Dir.pm > line 155 > # Path::Class::Dir::rmtree(Path::Class::Dir=HASH(0x1bbf140)) > called at t/06dir.t line 16 > # > # Looks like you failed 1 test of 33. > t/06dir.t ......... Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/33 subtests
See https://rt.cpan.org/Ticket/Display.html?id=105935 for a discussion about the File-Path change. There are votes (including mine) to revert this change in File-Path.
  Mon Sep 07 13:19:13 2015 The RT System itself - Status changed from 'new' to 'open'
  Mon Sep 07 14:21:26 2015 ether [...] cpan.org - Correspondence added
On 2015-09-07 10:19:12, SREZIC wrote: Show quoted text
> On 2015-09-07 09:25:08, RIBASUSHI wrote:
> > The change https://metacpan.org/source/RICHE/File-Path- > > 2.11/Changes#L4 > > trickles through Path::Class into t/06dir.t of this module, with the > > following result:
...and this is why I dropped all use of 'use warnings FATAL => "all"' and 'use Test::NoWarnings' in my tests last year, because it makes user installations fail due to no fault of the module itself. Warning tests should be author-only! use if $ENV{AUTHOR_TESTING}, 'Test::Warnings';
  Fri Oct 09 08:17:24 2015 ribasushi [...] leporine.io - Correspondence added
Sanity prevailed and a fixed File::Path just entered PAUSE. This issue no longer applies, though removing the check for 3rd party warnings is still worthwhile... at some point.
  Fri Oct 09 08:17:25 2015 ribasushi [...] leporine.io - Status changed from 'open' to 'resolved'