Skip Menu |

This queue is for tickets about the bignum CPAN distribution.

Report information
The Basics
Id: 119225
Status: resolved
Priority: 0/
Queue: bignum
People
Owner: Nobody in particular
Requestors: nanis [...] cpan.org
Cc: DBOOK [...] cpan.org
ether [...] cpan.org
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.47

History Show all quoted text
  Tue Dec 13 09:07:38 2016 nanis [...] cpan.org - Ticket created
Subject: Please fix Makefile.PL so it doesn't install over core version
Date: Tue, 13 Dec 2016 09:07:25 -0500
To: bug-bignum [...] rt.cpan.org
From: "A. Sinan Unur" <nanis [...] cpan.org>
perls since 5.12 look in site/lib first, therefore the upgrade has no effect. See also https://www.nntp.perl.org/group/perl.perl5.porters/2016/12/msg241637.html and https://twitter.com/sinan_unur/status/808650748334587905 HTH, -- Sinan
  Tue Dec 13 09:16:36 2016 nanis [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #119225] AutoReply: Please fix Makefile.PL so it doesn't install over core version
Date: Tue, 13 Dec 2016 09:16:27 -0500
To: bug-bignum [...] rt.cpan.org
From: "A. Sinan Unur" <nanis [...] cpan.org>
This should be considered a critical bug because, with perl version >= 5.12, subsequent upgrades will have no effect if user ever upgraded bignum and friends since installing perl. Anyone who installed 0.46 lost the ability to revert to the version that came with the original distribution simply by removing files from site/lib. -- Sinan
  Tue Dec 13 09:17:50 2016 nanis [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #119225] AutoReply: Please fix Makefile.PL so it doesn't install over core version
Date: Tue, 13 Dec 2016 09:17:40 -0500
To: bug-bignum [...] rt.cpan.org
From: "A. Sinan Unur" <nanis [...] cpan.org>
Of course, the same fix should also be applied to Makefile.PL in Math::BigInt Math::BigInt::FastCalc Math::BigRat I am not going to file extra bug reports for those. -- Sinan
  Tue Dec 13 10:47:14 2016 DBOOK [...] cpan.org - Cc DBOOK added
  Tue Dec 13 11:06:52 2016 DBOOK [...] cpan.org - Correspondence added
The correct fix for this is to include logic such as other dual life modules have, such that INSTALLDIRS is 'perl' only after the module became core and before 5.012: https://metacpan.org/source/EXODIST/Test-Simple-1.302067/Makefile.PL#L53-54 https://metacpan.org/source/JPEACOCK/version-0.9917/Makefile.PL#L63-65 For bignum and Math::BigRat, INSTALLDIRS should be 'perl' if ($] >= 5.008 and $] < 5.012). For Math::BigInt::FastCalc, INSTALLDIRS should be 'perl' if ($] >= 5.009003 and $] < 5.012). For Math::BigInt, INSTALLDIRS should be 'perl' if ($] < 5.012). In all other cases, it should be 'site' or omitted.
  Tue Dec 13 11:06:53 2016 The RT System itself - Status changed from 'new' to 'open'
  Tue Dec 13 12:23:27 2016 ether [...] cpan.org - Cc ETHER added
  Tue Dec 13 13:41:06 2016 peter.john.acklam [...] gmail.com - Correspondence added
I thought the first suggsted fix in CPAN RT #119199 would do the trick. I wasn't aware of the distiction < 5.12 vs. >= 5.12. I am working on a new release now.
  Tue Dec 13 16:12:55 2016 peter.john.acklam [...] gmail.com - Correspondence added
I have now uploaded the following files to CPAN. Hopefully, it is correct this time. bignum-0.47.tar.gz Math-BigInt-1.999806.tar.gz Math-BigInt-FastCalc-0.5005.tar.gz Math-BigRat-0.2611.tar.gz
  Tue Dec 13 17:17:27 2016 nanis [...] cpan.org - Correspondence added
CC: DBOOK [...] cpan.org, ether [...] cpan.org
Subject: Re: [rt.cpan.org #119225] Please fix Makefile.PL so it doesn't install over core version
Date: Tue, 13 Dec 2016 17:17:15 -0500
To: bug-bignum [...] rt.cpan.org
From: "A. Sinan Unur" <nanis [...] cpan.org>
On Tue, Dec 13, 2016 at 4:13 PM, Peter John Acklam via RT <bug-bignum@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=119225 > > > I have now uploaded the following files to CPAN. Hopefully, it is correct this time.
Seems to be OK on Windows with perl 5.24.1 RC4 C:\> cpan-outdated|cpanm --> Working on P/PJ/PJACKLAM/bignum-0.47.tar.gz Fetching https://cpan.metacpan.org/authors/id/P/PJ/PJACKLAM/bignum-0.47.tar.gz ... OK Configuring bignum-0.47 ... OK Building and testing bignum-0.47 ... OK Successfully installed bignum-0.47 --> Working on P/PJ/PJACKLAM/Math-BigInt-1.999806.tar.gz Fetching https://cpan.metacpan.org/authors/id/P/PJ/PJACKLAM/Math-BigInt-1.999806.tar.gz ... OK Configuring Math-BigInt-1.999806 ... OK Building and testing Math-BigInt-1.999806 ... OK Successfully installed Math-BigInt-1.999806 --> Working on P/PJ/PJACKLAM/Math-BigInt-FastCalc-0.5005.tar.gz Fetching https://cpan.metacpan.org/authors/id/P/PJ/PJACKLAM/Math-BigInt-FastCalc-0.5005.tar.gz ... OK Configuring Math-BigInt-FastCalc-0.5005 ... OK Building and testing Math-BigInt-FastCalc-0.5005 ... OK Successfully installed Math-BigInt-FastCalc-0.5005 --> Working on P/PJ/PJACKLAM/Math-BigRat-0.2611.tar.gz Fetching https://cpan.metacpan.org/authors/id/P/PJ/PJACKLAM/Math-BigRat-0.2611.tar.gz ... OK Configuring Math-BigRat-0.2611 ... OK Building and testing Math-BigRat-0.2611 ... OK Successfully installed Math-BigRat-0.2611 4 distributions installed C:\> perl -Mbignum -E "say $bignum::VERSION" 0.47 C:\> cpan-outdated --verbose C:\Users\sinan>
  Tue Dec 13 17:18:01 2016 nanis [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #119225] Please fix Makefile.PL so it doesn't install over core version
Date: Tue, 13 Dec 2016 17:17:45 -0500
To: bug-bignum [...] rt.cpan.org
From: "A. Sinan Unur" <nanis [...] cpan.org>
On Tue, Dec 13, 2016 at 4:13 PM, Peter John Acklam via RT <bug-bignum@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=119225 > > > I have now uploaded the following files to CPAN. Hopefully, it is correct this time.
Thank you very much for your work and quick response. -- Sinan
  Wed Dec 21 08:24:05 2016 peter.john.acklam [...] gmail.com - Status changed from 'open' to 'resolved'
  Wed Dec 21 08:24:11 2016 peter.john.acklam [...] gmail.com - Fixed in 0.47 added