Skip Menu | You are currently an anonymous guest. | Login | Return to Main | About rt.cpan.org
 

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.

X Report information
Id: 42442
Status: resolved
Worked: 30 min
Left: 0 min
Priority: 0/0
Queue: Spreadsheet-ParseExcel

Owner: Nobody
Requestors:
Cc:
AdminCc:

Severity: (no value)
Broken in: 0.45
Fixed in: (no value)



X History Display mode: Brief headersFull headers
#   Thu Jan 15 12:08:12 2009 cpan[...]pjedwards.co.uk - Ticket created  
Subject: Quick fix in Makefile.PL for VMS
[text/plain 736b]
Hello, and thanks again for working on Spreadsheet::ParseExcel.

I've a quick fix to allow the distribution to pass it's tests on VMS.
Makefile.PL has the following line:

test => {TESTS => 't/*.t t/examples/*t'}

The "t/examples" directory does not exist in the distribution and on
VMS I get:

Cannot determine source for t/examples/*t
at /perl_root/lib/Test/Harness.pm line 170
%SYSTEM-F-ABORT, abort
%MMK-F-ERRUPD, error status %X0000002C occurred when updating target
TEST_DYNAMIC

The fix is to change the line in Makefile.PL to:

test => {TESTS => 't/*.t'}

(or just remove it), I also noticed there are some files t/exmaples/*.t
that are commented out in the MANIFEST.

Cheers,

Peter (Stig) Edwards
#   Thu Jan 15 14:33:33 2009 JMCNAMARA - Correspondence added  
[text/plain 398b]
On Thu Jan 15 12:08:12 2009, cpan[...]pjedwards.co.uk wrote:
> Hello, and thanks again for working on Spreadsheet::ParseExcel.
>
> I've a quick fix to allow the distribution to pass it's tests on VMS.
> Makefile.PL has the following line:
>
> test => {TESTS => 't/*.t t/examples/*t'}

Hi Peter,

Thanks, I've fixed it in version control and it will be in the next release.

John.
--



#   Thu Jan 15 14:33:34 2009 RT_System - Status changed from 'new' to 'open'  
#   Tue Jan 20 04:20:07 2009 JMCNAMARA - Correspondence added 30 min  
[text/plain 48b]
Hi,

Fixed in version 0.46.

Thanks,

John.
--

#   Tue Jan 20 04:20:08 2009 JMCNAMARA - Status changed from 'open' to 'resolved'