Skip Menu |

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

Report information
The Basics
Id: 12147
Status: resolved
Priority: 0/
Queue: Test-Plan
People
Owner: geoff [...] cpan.org
Requestors: barbie [...] missbarbell.co.uk
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.02
Fixed in: (no value)

History Show all quoted text
  Tue Apr 05 09:58:20 2005 BARBIE [...] cpan.org - Ticket created
Subject: test scripts not listed correctly for Win32
The attached patch is specifically for Makefile.PL. In listing the test scripts to run, you use '*.t'. However, this doesn't always work very well on Win32, and it can be better to list the test scripts individually.
--- C:\wip\diffs\Test-Plan\Test-Plan-0.02/Makefile.PL Tue Mar 15 14:37:58 2005 +++ C:\wip\diffs\Test-Plan\Test-Plan-0.02-barbie/Makefile.PL Tue Apr 5 14:56:36 2005 @@ -7,11 +7,11 @@ use File::Find qw(find); use File::Spec (); -my %directories; +my %scripts; -find(sub { return unless m/\.t$/; $directories{$File::Find::dir} = 1 }, 't'); +find(sub { return unless m/\.t$/; $scripts{$File::Find::name} = 1 }, 't'); -my @tests = map { File::Spec->catfile($_, '*.t') } keys %directories; +my @tests = keys %scripts; WriteMakefile( NAME => 'Test::Plan',
  Tue Apr 05 10:43:13 2005 geoff [...] cpan.org - Status changed from 'new' to 'open'
  Tue Apr 05 10:43:13 2005 geoff [...] cpan.org - Correspondence added
[BARBIE - Tue Apr 5 09:58:20 2005]: Show quoted text
> The attached patch is specifically for Makefile.PL. In listing the > test scripts to run, you use '*.t'. However, this doesn't always > work very well on Win32, and it can be better to list the test > scripts individually.
your patch assumes that every file under t/ is a test file, which it may not be. but I'll try and work something up and include it in the next release. --Geoff
  Tue Apr 05 10:43:13 2005 geoff [...] cpan.org - Given to GEOFF
  Tue Apr 05 10:46:39 2005 geoff [...] cpan.org - Correspondence added
[GEOFF - Tue Apr 5 10:43:13 2005]: Show quoted text
> [BARBIE - Tue Apr 5 09:58:20 2005]: >
> > The attached patch is specifically for Makefile.PL. In listing the > > test scripts to run, you use '*.t'. However, this doesn't always > > work very well on Win32, and it can be better to list the test > > scripts individually.
> > your patch assumes that every file under t/ is a test file, which it may > not be. but I'll try and work something up and include it in the next > release.
uh, forget what I just said :) I clearly need more coffee... thanks for the patch. --Geoff
  Tue Mar 07 12:58:10 2006 geoff [...] cpan.org - Correspondence added
this was fixed in Test-Plan-0.03, with attribution given to another user who reported the problem 6 months after you did. sorry about that. --Geoff
  Tue Mar 07 12:58:11 2006 geoff [...] cpan.org - Status changed from 'open' to 'resolved'