Bug #75370 for DateTime-Util-Astro: Warning with perl 5.14

Tue Feb 28 05:54:30 2012 MAROS [...] cpan.org - Ticket created

Subject:

Warning with perl 5.14

I keep getting this warning with perl 5.14: Use of qw(...) as parentheses is deprecated at /opt/perl5.14.0/lib/site_perl/5.14.0/DateTime/Util/Astro/Common.pm line 322. The line "foreach my $f qw(longitude latitude elevation zone) {" should better be "foreach my $f (qw(longitude latitude elevation zone)) {"

Mon Sep 09 10:30:01 2013 PEPL [...] cpan.org - Severity Normal changed to Critical

Mon Sep 09 10:30:47 2013 PEPL [...] cpan.org - Correspondence added

Starting with Perl 5.18 the module will not compile any longer because of this, patch attached Am Di 28. Feb 2012, 05:54:30, MAROS schrieb: Show quoted text

> I keep getting this warning with perl 5.14: > Use of qw(...) as parentheses is deprecated at > /opt/perl5.14.0/lib/site_perl/5.14.0/DateTime/Util/Astro/Common.pm line 322. > > The line "foreach my $f qw(longitude latitude elevation zone) {" should > better be "foreach my $f (qw(longitude latitude elevation zone)) {"

Subject:

DateTime-Util-Astro-Common-RT75370.patch

--- lib/perl5/DateTime/Util/Astro/Common.pm.dist 2013-09-09 16:16:09.749044014 +0200 +++ lib/perl5/DateTime/Util/Astro/Common.pm 2013-09-09 16:16:32.802264037 +0200 @@ -319,7 +319,7 @@ BEGIN { - foreach my $f qw(longitude latitude elevation zone) { + foreach my $f (qw(longitude latitude elevation zone)) { eval sprintf( <<'EOM', $f, $f, $f); sub %s {

Mon Sep 09 10:30:47 2013 The RT System itself - Status changed from 'new' to 'open'

Mon Sep 09 18:17:00 2013 dmaki [...] cpan.org - Correspondence added

This module was pretty slow and sometimes incorrect because of all the round tripping between Perl and GMP and what not. I've moved all this stuff to DateTime::Astro, and if possible, would like to deprecate this module. So care to try that instead?

Mon Sep 23 08:33:34 2013 PEPL [...] cpan.org - Correspondence added

Am Mo 09. Sep 2013, 18:17:00, DMAKI schrieb: Show quoted text

> This module was pretty slow and sometimes incorrect because of all the > round tripping between Perl and GMP and what not. I've moved all this > stuff to DateTime::Astro, and if possible, would like to deprecate > this module. So care to try that instead?

https://metacpan.org/changes/distribution/DateTime-Event-Zodiac fixed that. Cheers to both of you, maros and dmaki! ~michael

Fri Apr 03 04:43:34 2015 SREZIC [...] cpan.org - Correspondence added

On 2013-09-09 18:17:00, DMAKI wrote: Show quoted text

> This module was pretty slow and sometimes incorrect because of all the > round tripping between Perl and GMP and what not. I've moved all this > stuff to DateTime::Astro, and if possible, would like to deprecate > this module. So care to try that instead?

There seems to be a remaining dependency on this module: DateTime-Event-Lunar-0.06 See https://metacpan.org/requires/distribution/DateTime-Util-Astro?sort=[[2,1]] (And DateTime-Event-Lunar also has a reverse dependency: DateTime-Indic-0.1)