Catalyst complains that an argument "isn't numeric in numeric eq (==)"

Bug #308508 reported by Eskild Hustvedt
2
Affects Status Importance Assigned to Milestone
Catalyst web-framework
Unknown
Unknown
libcatalyst-perl (Debian)
Fix Released
Unknown
libcatalyst-perl (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: libcatalyst-perl

When using the test server catalyst will complain a lot like this:
Argument "0,019022" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.
Argument "0,008011" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.
Argument "0,006365" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.
Argument "0,004394" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.
Argument "0,006583" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.
Argument "0,004566" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.
Argument "0,008730" isn't numeric in numeric eq (==) at /usr/share/perl5/Catalyst.pm line 1364.

As far as I am aware of, it is fixed upstream. The fix is very simple, patch attached. It removes a useless sprintf that converts the floating point number into an invalid string.

Revision history for this message
Eskild Hustvedt (zerodogg) wrote :
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

It doesn't seem like this has been fixed upstream. Current svn still shows the faulty line:

http://dev.catalystframework.org/svnweb/Catalyst/view/Catalyst-Runtime/5.80/trunk/lib/Catalyst.pm

Could you report it upstream? We can patch this for Ubuntu but indeed this should be done upstream.

Revision history for this message
Eskild Hustvedt (zerodogg) wrote :
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

Thanks. We might as well get 5.7015 in.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libcatalyst-perl - 5.7015-0ubuntu1

---------------
libcatalyst-perl (5.7015-0ubuntu1) jaunty; urgency=low

  * New upstream release:
    - Workaround change in LWP that broke a cookie test.
  * Removed unnecessary sprintf when using the test server:
    cause warnings on locales where commas are used for decimal markers
    (LP: #308508). Thanks to Eskild Hustvedt for the patch.

 -- Cesare Tirabassi <email address hidden> Tue, 16 Dec 2008 14:24:16 +0100

Changed in libcatalyst-perl:
status: New → Fix Released
Changed in libcatalyst-perl (Debian):
status: Unknown → Confirmed
Changed in libcatalyst-perl (Debian):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.