Skip Menu |

This queue is for tickets about the OLE-Storage_Lite CPAN distribution.

Report information
The Basics
Id: 34567
Status: resolved
Worked: 8 hours (480 min)
Priority: 0/
Queue: OLE-Storage_Lite

People
Owner: Nobody in particular
Requestors: andrew.benham [...] thus.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Problem with OLE-Storage_Lite-0.16
Date: Mon, 31 Mar 2008 11:11:26 +0100
To: bug-OLE-Storage_Lite [...] rt.cpan.org
From: Andrew Benham <andrew.benham [...] thus.net>
I've found a problem with OLE-Storage_Lite v0.16 when used with Math::BigInt v1.77 on Perl 5.6.1 Lines in OLE/Storage_Lite.pm state: 1327 my $iHSec = $iBigDt % 10000000; 1328 $iBigDt /= 10000000; 1329 my $iBigDay = int($iBigDt / (24*3600)) + 1; 1330 if ($iBigDay->numify eq $iBigDay) { 1331 $iBigDay = $iBigDay->numify; 1332 } 1333 my $iTime = int($iBigDt % (24*3600)); 1334 if ($iTime->numify eq $iTime) { 1335 $iTime = $iTime->numify; 1336 } This code works fine on Perl 5.8.8 - variables $iBigDay and $iTime are created as Math::BigInt objects. However on Perl 5.6.1 variables $iBigDay and $iTime are created as integers, and then the calls to the numify method blow up with """Can't call method "numify" without a package or object reference""" I can fix the problem by removing the 'int()' cast in two places - then variables $iBigDay and $iTime are created as Math::BigInt objects on both Perl 5.6.1 and 5.8.8 I haven't done exhaustive testing though to prove the results are always the same. Perhaps using the bfloor method would be a better idea ? I don't know if a later Math::BigInt would help, but 1.80 onwards need Perl 5.6.2 as a minimum. -- Andrew Benham Demon@Thus andrew.benham@thus.net Finchley, London N3 2QQ, U.K. Tel: 020 8495 6343 Fax: 020 8495 6037
On Mon Mar 31 06:16:47 2008, andrew.benham@thus.net wrote: Show quoted text
> I've found a problem with OLE-Storage_Lite v0.16 when used with > Math::BigInt v1.77 on Perl 5.6.1
Hi Andrew, It looks this is caused by a patch that went into version 0.15: http://rt.cpan.org/Ticket/Display.html?id=31006 I'll take a look at this issue and in the meantime you could revert to 0.14 as a workaround (unless you are using HP-UX which had a fix in 0.16). Btw, what OS are using. Also are you using the module in conjunction with Spreadsheet::WriteExcel, Spreadsheet::ParseExcel or some other module? John. --
Subject: Re: [rt.cpan.org #34567] Problem with OLE-Storage_Lite-0.16
Date: Mon, 31 Mar 2008 15:15:30 +0100
To: bug-OLE-Storage_Lite [...] rt.cpan.org
From: Andrew Benham <andrew.benham [...] thus.net>
John McNamara via RT wrote: Show quoted text
> It looks this is caused by a patch that went into version 0.15: > > http://rt.cpan.org/Ticket/Display.html?id=31006 > > I'll take a look at this issue and in the meantime you could revert to > 0.14 as a workaround (unless you are using HP-UX which had a fix in 0.16). > > Btw, what OS are using. Also are you using the module in conjunction > with Spreadsheet::WriteExcel, Spreadsheet::ParseExcel or some other module?
Sorry, should have said we're experiencing this on Solaris-9 (both sparc and x86) with the vendor's Perl 5.6.1, and on Solaris-8 sparc with our own build of Perl 5.6.1 to match that on Solaris-9. We have both Spreadsheet::WriteExcel and Spreadsheet::ParseExcel in use. -- Andrew Benham Demon@Thus andrew.benham@thus.net Finchley, London N3 2QQ, U.K. Tel: 020 8495 6343 Fax: 020 8495 6037
On Mon Mar 31 06:16:47 2008, andrew.benham@thus.net wrote: Show quoted text
> I've found a problem with OLE-Storage_Lite v0.16 when used with > Math::BigInt v1.77 on Perl 5.6.1
Hi, I've replaced the Math::BigInt usage in OLE::Storage_Lite with other methods in version 0.17. You can test it on your system when you get a chance. There is now a dedicated test for the internal date handling routines as well so if it passes 'make test' you should be okay. John. --
Subject: Re: [rt.cpan.org #34567] Problem with OLE-Storage_Lite-0.16
Date: Mon, 12 May 2008 16:37:07 +0100
To: bug-OLE-Storage_Lite [...] rt.cpan.org
From: Andrew Benham <andrew.benham [...] thus.net>
John McNamara via RT wrote: Show quoted text
> I've replaced the Math::BigInt usage in OLE::Storage_Lite with other > methods in version 0.17. > > You can test it on your system when you get a chance. There is now a > dedicated test for the internal date handling routines as well so if it > passes 'make test' you should be okay.
Builds and passes tests on all our platforms. Many thanks. -- Andrew Benham Demon@Thus andrew.benham@thus.net Finchley, London N3 2QQ, U.K. Tel: 020 8495 6343 Fax: 020 8495 6037
Subject: Re: [rt.cpan.org #34567] Problem with OLE-Storage_Lite-0.16
Date: Mon, 12 May 2008 19:42:27 +0100
To: bug-OLE-Storage_Lite [...] rt.cpan.org
From: "John McNamara" <jmcnamara13 [...] gmail.com>
On Mon, May 12, 2008 at 4:37 PM, andrew.benham@thus.net via RT < bug-OLE-Storage_Lite@rt.cpan.org> wrote: Show quoted text
> > Builds and passes tests on all our platforms. Many thanks. >
Hi Andrew, Thanks for that. I'll close the issue and you can reopen it if there any any further problems. John. --
Fixed in version 0.17