-----Original Message----- From: Ingo Schmid

finally I studied how to create and upload modules to CPAN. So I just
uploaded PDL::IO::Nifti to PAUSE. I would have liked to test the
download right away, but I guess it takes a few ours to appear.

First place it will show up is on backpan ( http://backpan.perl.org/authors/id/F/FA/FANTASMA/ ). I reckon that usually happens within a minute, though I haven't actually timed it. That's probably no use if you want to use CPAN.pm (or the like) to download it, as I don't think those modules will grab anything from backpan.

Any
feedback would be welcome.

The -T switch in 00-load.t prevents that script from running for me:

t/00-load.t .......
#   Failed test 'use PDL::IO::Nifti;'
t/00-load.t ....... 1/? #   at t/00-load.t line 10.
#     Tried to use 'PDL::IO::Nifti'.
# Error: Insecure $ENV{PATH} while running with -T switch at C:/MinGW/perl516/site/lib/PDL/IO/Pic.pm line 190, <DATA> line 207.
# Compilation failed in require at (eval 5) line 21, <DATA> line 207.
# BEGIN failed--compilation aborted at (eval 5) line 21, <DATA> line 207.
# BEGIN failed--compilation aborted at C:\sisyphusion\PDL-IO-Nifti-0.52\blib\lib/PDL/IO/Nifti.pm line 6, <DATA> line 207.
# Compilation failed in require at (eval 4) line 2, <DATA> line 207.
# BEGIN failed--compilation aborted at (eval 4) line 2, <DATA> line 207.
Bailout called.  Further testing stopped:
# Looks like you failed 1 test of 1.
# Looks like your test exited with 255 just after 1.
FAILED--Further testing stopped.
dmake:  Error code 255, while making 'test_dynamic'

It's rather unusual to have the '-T' switch in test scripts. I would remove it from those files.

Ultimately, it would be a good thing to include some test scripts that verify that the module's functions are behaving as expected. The test suite doesn't currently do that - it verifies little more than that the module is loadable..

And I would probably begin the Makefile.PL with:
use 5.10.0;
or
use 5.010;

It currently specifies 5.006 - and the 5.10.0 dependency therefore doesn't get detected until the 'make test' stage. (Fixing that will turn a FAIL into an NA with cpan-testers running a pre-5.10 perl)

Cheers,
Rob

_______________________________________________
Perldl mailing list
[email protected]
http://mailman.jach.hawaii.edu/mailman/listinfo/perldl

Reply via email to