On Tue, Jan 21, 2014 at 8:12 PM, <[email protected]> wrote: > > > -----Original Message----- From: Chris Marshall > Sent: Wednesday, January 22, 2014 6:23 AM > To: [email protected] > Subject: [Perldl] CHM/PDL-IO-HDF5-0.63_01.tar.gz released to CPAN > > >> Announcing the release of the latest PDL::IO::HDF5 as >> a CPAN developers release. > > > There's a couple of issues with my Windows builds. > > On my 32-bit-integer builds of perl: > > t/attribPDL.t ..... Invalid type 'q' in pack at Basic/Core/Core.pm.PL (i.e. > PDL::Core.pm) line 1318. > # Looks like your test exited with 255 before it could output anything. > t/attribPDL.t ..... Dubious, test returned 255 (wstat 65280, 0xff00) > Failed 13/13 subtests
This looks like a bug in the longlong support since if the perl does not support 64bit ints, then it doesn't support the qQ pack options. Maybe you could track this down. > All other tests pass. > > On my 64-bit-integer builds of perl, all tests pass, but I get the following > warning: > > HDF5.xs:2164:9: warning: overflow in implicit constant conversion > [-Woverflow] It looks like this value should be a size_t type rather than int. If I make the change it seems to work here. I've attached a patchfile with that change. Does it work? Thanks, Chris > I mention this in case the warning signifies a potential for trouble. > Here's the relevant XS code - I've marked line 2164 as "<<HERE": > > ################################################# > # Sub to add the H5T_VARIABLE constant > # This is added manually here, rather than regenerate the constant function > above > int > H5T_VARIABLE() > CODE: > RETVAL = H5T_VARIABLE; <<HERE > OUTPUT: > RETVAL > > ################################################# > > Let me know if you want a bug report filed on rt for any of this. > > Cheers, > Rob
0001-Make-H5T_VARIABLE-constant-return-a-size_t-value.patch
Description: Binary data
_______________________________________________ Perldl mailing list [email protected] http://mailman.jach.hawaii.edu/mailman/listinfo/perldl
