Skip Menu |
 

This queue is for tickets about the JavaScript-V8 CPAN distribution.

Report information
The Basics
Id: 73963
Status: open
Priority: 0/
Queue: JavaScript-V8

People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in:
  • 0.06
  • 0.07
Fixed in: (no value)



Subject: Fails on nvsize==16
Download (untitled) / with headers
text/plain 473b
Sample fail report: http://www.cpantesters.org/cpan/report/17469128 This is a floating point issue. The test seems to compare floating point numbers for string-equality. This often fails when the hardware architecture is dfferent, in this case nvsize==16. BTW, out of curiosity: I have no idea why this distribution received only such a small amount of tests from cpantesters (at the time of this writing I count 22). Do you have an explanation? HTH, Thanks && Regards,
Download (untitled) / with headers
text/plain 745b
On Thu Jan 12 16:55:26 2012, ANDK wrote: Show quoted text
> Sample fail report: > > http://www.cpantesters.org/cpan/report/17469128 > > This is a floating point issue. The test seems to compare floating point > numbers for string-equality. This often fails when the hardware > architecture is dfferent, in this case nvsize==16.
Sounds likely, will look into it. Show quoted text
> BTW, out of curiosity: I have no idea why this distribution received > only such a small amount of tests from cpantesters (at the time of this > writing I count 22). Do you have an explanation?
I suspect because it requires v8 + headers to be installed, which isn't that common. I have a todo to make it pull a copy in if the system doesn't have v8 (via Alien::V8) but haven't got to it yet.
Download (untitled) / with headers
text/plain 109b
Pull request fixes the broeken tests: https://github.com/dgl/javascript-v8/pull/10 Hope you like it, Enjoy!


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

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