Bug #79658 for Chart: defined(@array) is deprecated in Perl 5.16

Fri Sep 14 20:13:12 2012 lpsolit [...] gmail.com - Ticket created

Subject:	defined(@array) is deprecated in Perl 5.16
Date:	Sat, 15 Sep 2012 02:12:56 +0200
To:	bug-Chart [...] rt.cpan.org
From:	"Frédéric Buclin" <lpsolit [...] gmail.com>

When using Chart::Base with Perl 5.16.1, we get: defined(@array) is deprecated at lib/Chart/Base.pm line 181. (Maybe you should just omit the defined()?) defined(@array) is deprecated at lib/Chart/Base.pm line 233. (Maybe you should just omit the defined()?) The culprit lines are: @data = @$rdata if defined @$rdata; It should be: @data = @$rdata if @$rdata; or: @data = @$rdata if scalar @$rdata;

Sun Sep 16 14:09:08 2012 tmannerm [...] cpan.org - Cc TMANNERM added

Fri Nov 07 17:56:34 2014 SREZIC [...] cpan.org - Correspondence added

On 2012-09-14 20:13:12, lpsolit@gmail.com wrote: Show quoted text

> When using Chart::Base with Perl 5.16.1, we get: > > defined(@array) is deprecated at lib/Chart/Base.pm line 181. > (Maybe you should just omit the defined()?) > defined(@array) is deprecated at lib/Chart/Base.pm line 233. > (Maybe you should just omit the defined()?) > > The culprit lines are: > > @data = @$rdata if defined @$rdata; > > It should be: > > @data = @$rdata if @$rdata; > > or: > > @data = @$rdata if scalar @$rdata;

With even newer perls (5.21.x) this is not anymore a warning, but an error.

Fri Nov 07 17:56:34 2014 The RT System itself - Status changed from 'new' to 'open'

Sun Nov 09 05:19:34 2014 chart [...] fs.wettzell.de - Correspondence added

Hello, Chart Version 2.4.8 has resolved this issue. Please, download it Chart-2.4.8 and test it. I would appreciate a report. Tnanks Chartgrp Am Fr 14. Sep 2012, 20:13:12, lpsolit@gmail.com schrieb: Show quoted text

> When using Chart::Base with Perl 5.16.1, we get: > > defined(@array) is deprecated at lib/Chart/Base.pm line 181. > (Maybe you should just omit the defined()?) > defined(@array) is deprecated at lib/Chart/Base.pm line 233. > (Maybe you should just omit the defined()?) > > The culprit lines are: > > @data = @$rdata if defined @$rdata; > > It should be: > > @data = @$rdata if @$rdata; > > or: > > @data = @$rdata if scalar @$rdata;

Sun Nov 09 05:19:36 2014 chart [...] fs.wettzell.de - Status changed from 'open' to 'resolved'

Sun Nov 09 05:19:38 2014 chart [...] fs.wettzell.de - Correspondence added

Hello, Chart Version 2.4.8 has resolved this issue. Please, download it Chart-2.4.8 and test it. I would appreciate a report. Tnanks Chartgrp Am Fr 14. Sep 2012, 20:13:12, lpsolit@gmail.com schrieb: Show quoted text

> When using Chart::Base with Perl 5.16.1, we get: > > defined(@array) is deprecated at lib/Chart/Base.pm line 181. > (Maybe you should just omit the defined()?) > defined(@array) is deprecated at lib/Chart/Base.pm line 233. > (Maybe you should just omit the defined()?) > > The culprit lines are: > > @data = @$rdata if defined @$rdata; > > It should be: > > @data = @$rdata if @$rdata; > > or: > > @data = @$rdata if scalar @$rdata;

Tue Dec 02 19:27:51 2014 dylan [...] hardison.net - Correspondence added

2.4.8 has not been released it seems.

Tue Dec 02 19:27:53 2014 dylan [...] hardison.net - Status changed from 'resolved' to 'open'

Wed Dec 03 02:03:58 2014 dassing [...] fs.wettzell.de - Correspondence added

Subject:	Re: [rt.cpan.org #79658] defined(@array) is deprecated in Perl 5.16
Date:	Wed, 03 Dec 2014 08:03:33 +0100
To:	bug-Chart [...] rt.cpan.org
From:	"R.Dassing" <dassing [...] fs.wettzell.de>

Hello, the URL http://search.cpan.org/~chartgrp/Chart-2.4.8/ shows the release. Regards, Chartgrp Am 03.12.2014 um 01:27 schrieb Dylan William Hardison via RT: Show quoted text

> Queue: Chart > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=79658 > > > 2.4.8 has not been released it seems. >

Download smime.p7s
application/pkcs7-signature 4.6k

Message body not shown because it is not plain text.

Sat Jan 17 11:50:28 2015 lpsolit [...] gmail.com - Correspondence added

Subject:	[rt.cpan.org #79658] defined(@array) is deprecated in Perl 5.16
Date:	Sat, 17 Jan 2015 17:50:13 +0100
To:	bug-Chart [...] rt.cpan.org
From:	"Frédéric Buclin" <lpsolit [...] gmail.com>

http://search.cpan.org/~chartgrp/Chart-2.4.8/ indeed shows the release, but when using 'cpan' to install it, you get: cpan[1]> install Chart::Lines Reading '/root/.cpan/Metadata' Database was generated on Sat, 17 Jan 2015 16:29:02 GMT Chart::Lines is up to date (2.4.6). It reports 2.4.6 as being the latest release, not 2.4.8. So it looks like 2.4.8 hasn't been pushed correctly to CPAN.

Mon Feb 02 12:50:26 2015 dylan [...] hardison.net - Correspondence added

On Sat Jan 17 11:50:28 2015, lpsolit@gmail.com wrote: Show quoted text

> It reports 2.4.6 as being the latest release, not 2.4.8. So it looks > like 2.4.8 hasn't been pushed correctly to CPAN.

There is something wrong with the dist, I'm investigating. The most telling sign is that the dist does not show up in https://cpan.metacpan.org/modules/02packages.details.txt

Mon Feb 02 13:31:07 2015 ether [...] cpan.org - Correspondence added

On 2015-02-02 09:50:26, DHARDISON wrote: Show quoted text

> On Sat Jan 17 11:50:28 2015, lpsolit@gmail.com wrote:

> > It reports 2.4.6 as being the latest release, not 2.4.8. So it looks > > like 2.4.8 hasn't been pushed correctly to CPAN.

> > There is something wrong with the dist, I'm investigating. The most > telling sign is that the dist does not show up in > https://cpan.metacpan.org/modules/02packages.details.txt

The key is in the email that PAUSE sent back to the author when the tarball was uploaded -- but only the author (and the PAUSE admins) can view that. Issues that I can see: - there is no Chart.pm (PAUSE now requires that there be a module that matches the name of the dist) - many other tooling issues as listed here: http://cpants.cpanauthors.org/dist/Chart

Mon Feb 02 13:34:25 2015 ether [...] cpan.org - Cc ETHER added

Tue Feb 03 07:46:17 2015 chart [...] fs.wettzell.de - Correspondence added

Subject:	Re: [rt.cpan.org #79658] defined(@array) is deprecated in Perl 5.16
Date:	Tue, 03 Feb 2015 13:45:47 +0100
To:	bug-Chart [...] rt.cpan.org
From:	chart <chart [...] fs.wettzell.de>

Hello Karen, by using the program 'cpan' I recognized that the MD5 checksums did not match for the last upload. I will retry it with a new version within the next days. I hope this will solve the problem. Regards Reiner CHARTGRP Am 02.02.2015 um 19:31 schrieb Karen Etheridge via RT: Show quoted text

> Queue: Chart > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=79658 > > > On 2015-02-02 09:50:26, DHARDISON wrote:

>> On Sat Jan 17 11:50:28 2015, lpsolit@gmail.com wrote:

>>> It reports 2.4.6 as being the latest release, not 2.4.8. So it looks >>> like 2.4.8 hasn't been pushed correctly to CPAN.

>> There is something wrong with the dist, I'm investigating. The most >> telling sign is that the dist does not show up in >> https://cpan.metacpan.org/modules/02packages.details.txt

> The key is in the email that PAUSE sent back to the author when the tarball was uploaded -- but only the author (and the PAUSE admins) can view that. > > Issues that I can see: > - there is no Chart.pm (PAUSE now requires that there be a module that matches the name of the dist) > - many other tooling issues as listed here: http://cpants.cpanauthors.org/dist/Chart

Tue Feb 03 07:46:44 2015 chart [...] fs.wettzell.de - Correspondence added

Subject:	Re: [rt.cpan.org #79658] defined(@array) is deprecated in Perl 5.16
Date:	Tue, 03 Feb 2015 13:46:19 +0100
To:	bug-Chart [...] rt.cpan.org
From:	chart <chart [...] fs.wettzell.de>

Hello Dylan William Hardison, by using the program 'cpan' I recognized that the MD5 checksums did not match for the last upload. I will retry it with a new version within the next days. I hope this will solve the problem. Regards Reiner CHARTGRP Am 02.02.2015 um 18:50 schrieb Dylan William Hardison via RT: Show quoted text

> Queue: Chart > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=79658 > > > On Sat Jan 17 11:50:28 2015, lpsolit@gmail.com wrote:

>> It reports 2.4.6 as being the latest release, not 2.4.8. So it looks >> like 2.4.8 hasn't been pushed correctly to CPAN.

> There is something wrong with the dist, I'm investigating. The most telling sign is that the dist does not show up in https://cpan.metacpan.org/modules/02packages.details.txt >

Sat Feb 07 07:00:11 2015 lpsolit [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #79658] defined(@array) is deprecated in Perl 5.16
Date:	Sat, 07 Feb 2015 12:59:42 +0100
To:	bug-Chart [...] rt.cpan.org
From:	"Frédéric Buclin" <lpsolit [...] gmail.com>

Chart 2.4.9 doesn't fix the problem. http://www.cpan.org/modules/02packages.details.txt still lists 2.4.6 as being the latest version.

Sat Feb 07 13:50:21 2015 ether [...] cpan.org - Correspondence added

RT-Send-CC:

dassing [...] fs.wettzell.de, chart [...] fs.wettzell.de

On 2015-02-07 04:00:11, lpsolit@gmail.com wrote: Show quoted text

> Chart 2.4.9 doesn't fix the problem. > > http://www.cpan.org/modules/02packages.details.txt still lists 2.4.6 as > being the latest version.

Chart-2.4.9 does not contain a Chart.pm, which is necessary for proper PAUSE indexing. This error should have been evident in the email that PAUSE sends after processing the upload -- did you (the author) get the email?

Mon Feb 09 02:23:27 2015 chart [...] fs.wettzell.de - Correspondence added

Subject:	Re: [rt.cpan.org #79658] defined(@array) is deprecated in Perl 5.16
Date:	Mon, 09 Feb 2015 08:23:02 +0100
To:	bug-Chart [...] rt.cpan.org
From:	chart <chart [...] fs.wettzell.de>

Hi, The upload of new module Chart failed. This problem was reported to modules@perl.org. I got the following answer from Neil Browers (modules@perl.org): The 'problem' is that PAUSE now considers package names case-insensitively when resolving permissions. The Finance::Quant module defines a package ‘chart’, so SANTEX got ownership of ‘chart’, and now when you try to upload something with ‘Chart’, it’s the same permission that’s being checked. And because someone else has permission for that package, the indexing step fails. I’ll raise an issue on Finance-Quant, and ask Hagen to refactor his dist to use package names under Finance::Quant::*. Once he’s done that he’ll need to delete all releases from CPAN that contain the ‘chart’ package, then free up the permission or transfer it to you. Please bear with us while I try to get this resolved as quickly as possible. Neil Am 07.02.2015 um 13:00 schrieb Frédéric Buclin via RT: Show quoted text

> Queue: Chart > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=79658 > > > Chart 2.4.9 doesn't fix the problem. > > http://www.cpan.org/modules/02packages.details.txt still lists 2.4.6 as > being the latest version. > >