Bug 1412052
| Summary: | perl Math::BigInt regression in ignoring white space with non-decimal numbers | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Stewart Smith <stewart> | ||||
| Component: | perl-Math-BigInt | Assignee: | Petr Pisar <ppisar> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | high | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | 24 | CC: | perl-devel, ppisar | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | perl-Math-BigInt-1.9998.08-1.fc26 perl-Math-BigInt-1.9997.27-2.fc25 perl-Math-BigInt-1.9997.15-3.fc24 | Doc Type: | If docs needed, set a value | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2017-01-14 03:51:40 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
You are right. But I cannot find the report at upstream <https://rt.cpan.org/Public/Dist/Display.html?Name=Math-BigInt>. Can you point me to your upstream bug report? Created attachment 1239362 [details]
Proposed fix
I found you upstream bug now. I posted the fix there. Upstream accepted the fix with different tests. perl-Math-BigInt-1.9997.27-2.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-3384698335 perl-Math-BigInt-1.9997.15-3.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6fdbd25659 perl-Math-BigInt-1.9997.15-3.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-6fdbd25659 perl-Math-BigInt-1.9997.27-2.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-3384698335 perl-Math-BigInt-1.9997.27-2.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report. perl-Math-BigInt-1.9997.15-3.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report. |
Description of problem: The Math::BigInt manual states that leading and trailing whitespace is ignored, and in fact this has been the behaviour for many years. However, in recent Math::BigInt (that has shipped in Linux distributions... e.g. Fedora 25), this behaviour is broken, which breaks our tools that rely on this. With Math::BigInt 1.999727: 'use Math::BigInt; print Dumper(Math::BigInt->new("0x10\n\t\t\t"));'; $VAR1 = bless( { 'sign' => 'NaN', 'value' => [ 0 ] }, 'Math::BigInt' ); With Math::BigInt 1.9993: 'use Math::BigInt; print Dumper(Math::BigInt->new("0x10\n\t\t\t"));'; $VAR1 = bless( { 'sign' => '+', 'value' => [ 16 ] }, 'Math::BigInt' ); The first version to exhibit this buggy behaviour was 1.999712, with 1.999710 and 1.9997_11 both not having the bug. All versions after 1.999712, including the latest, 1.999807 exhibit the bug. Version-Release number of selected component (if applicable): perl-Math-BigInt-1.9997.27-1.fc25.noarch How reproducible: 100% Steps to Reproduce: 1. Math::BigInt->new("0x10\n\t\t\t") Actual results: print Dumper(Math::BigInt->new("0x10\n\t\t\t"));'; $VAR1 = bless( { 'sign' => 'NaN', 'value' => [ 0 ] }, 'Math::BigInt' ); Expected results: 'use Math::BigInt; print Dumper(Math::BigInt->new("0x10\n\t\t\t"));'; $VAR1 = bless( { 'sign' => '+', 'value' => [ 16 ] }, 'Math::BigInt' ); Additional info: I've reported this upstream, so I'm largely filing this as a Fedora bug so that it's tracked for Fedora - as this means you cannot compile OpenPOWER firmware on Fedora 25.