search

stevieb9 / ipc-shareable

Share Perl variables across processes and scripts
GNU General Public License v2.0
3 stars 2 forks source link

Double size is not compatible (-Duselongdouble?) #5

Closed eserte closed 3 years ago

eserte commented 3 years ago

On some of my smokers the test suite fails like this:

Double size is not compatible at /opt/perl-5.26.2D/lib/site_perl/5.26.2/x86_64-linux-ld/Storable.pm line 471, at /home/cpansand/.cpan/build/2021061006/IPC-Shareable-1.00-3/blib/lib/IPC/Shareable.pm line 546.
t/71-unspawn.t .......... 
Dubious, test returned 255 (wstat 65280, 0xff00)
No subtests run

This is a perl compiled with -Duselongdouble. I can image that it's also failing on quadmath perls (but I don't have any of these available).

stevieb9 commented 3 years ago

Will compile a version with -Duselongdouble and see what's happening here. Don't yet know if this problem is with this software, or Storable

eserte commented 3 years ago

I guess that a Storable-serialized float (NV) has the double size when created with a uselongdouble perl. I think the easiest is to just skip the test for -Duselongdouble perls, or in general, for all perls with $Config{nvsize}!=8.

stevieb9 commented 3 years ago

Thanks for that. I'll get that implemented.

On Fri, Jun 25, 2021 at 8:36 AM Slaven Rezić @.***> wrote:

I guess that a Storable-serialized float (NV) has the double size when created with a uselongdouble perl. I think the easiest is to just skip the test for -Duselongdouble perls, or in general, for all perls with $Config{nvsize}!=8.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/stevieb9/ipc-shareable/issues/5#issuecomment-868626386, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADDQEP5P3546BD26QZCLZ23TUSO7TANCNFSM46SSV7DQ .

stevieb9 commented 3 years ago

Fixed in https://github.com/stevieb9/ipc-shareable/commit/8727edd2a1bf6a1f0fdfdebe8149724fc8cdbee5