Bug 196773 - math/p5-Math-BigInt: overloading .. fails
Summary: math/p5-Math-BigInt: overloading .. fails
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Kurt Jaeger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-15 18:28 UTC by papowell
Modified: 2016-02-07 17:42 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (tobez)


Attachments
SHAR of modified p5-Math-BigInt port (2.75 KB, text/plain)
2015-01-15 18:28 UTC, papowell
no flags Details
patch (1.09 KB, patch)
2016-02-07 17:33 UTC, Kurt Jaeger
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description papowell 2015-01-15 18:28:22 UTC
Created attachment 151693 [details]
SHAR of modified p5-Math-BigInt port

The Math::BigInt - Arbitrary size integer math package tries to overload the '..' operator.  This is not allowed/supported by Perl 5.18 (and earlier, I suspect).  In the BigInt.pm module there are comments to this effect.

This patch comments out the overload '..' operation.  If overloading of '..' is allowed at some point then this issue may need to be revisited.

Attached is a SHAR file of the modified p5-Math-BigInt port.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2015-01-15 18:28:22 UTC
Auto-assigned to maintainer tobez@FreeBSD.org
Comment 2 Kurt Jaeger freebsd_committer freebsd_triage 2016-02-07 17:33:27 UTC
Created attachment 166715 [details]
patch

Updated patch. lang/perl5* provides its own bigint module, with that patch already included. So here's the one for the seperate module.
Comment 3 commit-hook freebsd_committer freebsd_triage 2016-02-07 17:41:58 UTC
A commit references this bug:

Author: pi
Date: Sun Feb  7 17:41:51 UTC 2016
New revision: 408433
URL: https://svnweb.freebsd.org/changeset/ports/408433

Log:
  math/p5-Math-BigInt: the overload of '..' is still not supported in perl

  - Similar patch is done in the lang/perl* ports

  PR:		196773
  Submitted by:	papowell@astart.com
  Approved by:	tobez (maintainer timeout)

Changes:
  head/math/p5-Math-BigInt/Makefile
  head/math/p5-Math-BigInt/files/
  head/math/p5-Math-BigInt/files/patch-lib_Math_BigInt.pm
Comment 4 Kurt Jaeger freebsd_committer freebsd_triage 2016-02-07 17:42:14 UTC
Committed, thanks.