Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the DateTime-TimeZone CPAN distribution.

Report information
The Basics
Id: 17539
Status: resolved
Priority: 0/
Queue: DateTime-TimeZone
People
Owner: Nobody in particular
Requestors: ljacobs [...] 10east.com
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in:
  • 0.38
  • 0.40
  • 0.41
Fixed in: 0.43

History Show all quoted text
  Wed Feb 08 12:25:33 2006 Guest - Ticket created
Subject: TimeZoneCatalog.pm does not contain AKST9AKDT
The valid time zone identifier AKST9AKDT (for Alaska) does not exist in TimeZoneCatalog.pm, which causes this simple test: perl -MData::Dumper -MDateTime -MDateTime::TimeZone -e '$tz = "AKST9AKDT"; $a = DateTime::TimeZone->new("name" => $tz); print &Dumper(\$a), "\n"' to fail. With this simple patch: --- ./TimeZoneCatalog.pm 2006-02-08 12:05:13.000000000 -0500 +++ DateTime/TimeZoneCatalog.pm 2006-02-07 17:55:00.000000000 -0500 @@ -793,6 +793,7 @@ %DateTime::TimeZone::LINKS = ( + 'AKST9AKDT' => 'America/Anchorage', 'Africa/Timbuktu' => 'Africa/Bamako', 'America/Argentina/ComodRivadavia' => 'America/Argentina/Catamarca', 'America/Atka' => 'America/Adak', that same test will now return a large hash. The diff was determined by comparing the version of TimeZoneCatalog.pm that Debian distributes with that from Gentoo. Apparently, Debian patches that file themselves. I discovered this because we have a customer in Alaska, and code to display the proper local time on their web interface works on all the Debian boxes in the office, but did not on my Gentoo box.
  Mon Feb 20 22:59:10 2006 autarch [...] urth.org - Correspondence added
Subject: Re: [rt.cpan.org #17539] TimeZoneCatalog.pm does not contain AKST9AKDT
Date: Mon, 20 Feb 2006 21:58:45 -0600 (CST)
To: Guest via RT <bug-DateTime-TimeZone [...] rt.cpan.org>
From: Dave Rolsky <autarch [...] urth.org>
On Wed, 8 Feb 2006, Guest via RT wrote: Show quoted text
> perl -MData::Dumper -MDateTime -MDateTime::TimeZone -e '$tz = > "AKST9AKDT"; $a = DateTime::TimeZone->new("name" => $tz); print > &Dumper(\$a), "\n"' > > to fail. With this simple patch: > > --- ./TimeZoneCatalog.pm 2006-02-08 12:05:13.000000000 -0500 > +++ DateTime/TimeZoneCatalog.pm 2006-02-07 17:55:00.000000000 -0500 > @@ -793,6 +793,7 @@ > > %DateTime::TimeZone::LINKS = > ( > + 'AKST9AKDT' => 'America/Anchorage', > 'Africa/Timbuktu' => 'Africa/Bamako', > 'America/Argentina/ComodRivadavia' => 'America/Argentina/Catamarca', > 'America/Atka' => 'America/Adak', > > that same test will now return a large hash. The diff was determined by > comparing the version of TimeZoneCatalog.pm that Debian distributes > with that from Gentoo. Apparently, Debian patches that file themselves. > I discovered this because we have a customer in Alaska, and code to > display the proper local time on their web interface works on all the > Debian boxes in the office, but did not on my Gentoo box.
This will be in the next release. It might've gone in sooner if the Debian maintainer had done his/her job and sent the patch to me (grr). -dave /*=================================================== VegGuide.Org www.BookIRead.com Your guide to all that's veg. My book blog ===================================================*/
  Mon Feb 20 22:59:11 2006 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 21 10:10:46 2006 ljacobs [...] 10east.com - Correspondence added
Subject: Re: [rt.cpan.org #17539] TimeZoneCatalog.pm does not contain AKST9AKDT
Date: Tue, 21 Feb 2006 10:09:50 -0500
To: bug-DateTime-TimeZone [...] rt.cpan.org
From: Lance Jacobs <ljacobs [...] 10east.com>
On Mon, 2006-02-20 at 22:59 -0500, autarch@urth.org via RT wrote: [snip] Show quoted text
> This will be in the next release. It might've gone in sooner if the Debian > maintainer had done his/her job and sent the patch to me (grr).
Dave, Thanks for letting me know. Since it appears that this was a Debian patch that was not passed along to you, I'd like to let you know about one other difference that we found between Debian's and Gentoo's version of TimeZoneCatalog.pm (based on version 0.38): --- TimeZoneCatalog.pm-0.38 2006-02-07 17:55:08.000000000 -0500 +++ TimeZoneCatalog.pm-Debian 2006-02-21 09:41:31.000000000 -0500 @@ -908,5 +908,5 @@ 'US/Arizona' => 'America/Phoenix', 'US/Central' => 'America/Chicago', - 'US/East-Indiana' => 'America/Indiana/Indianapolis', + 'US/East-Indiana' => 'America/Indianapolis', 'US/Eastern' => 'America/New_York', 'US/Hawaii' => 'Pacific/Honolulu', I did not mention that one before as I am not sure what all the implications of that change are, or why they might have done it, but there it is. Regards, -- Lance Jacobs Email: ljacobs@10east.com Senior Open Systems Engineer Phone: 904-220-3627 10 East Corp. FAX: 904-384-1038
  Tue Feb 21 15:12:01 2006 autarch [...] urth.org - Correspondence added
Subject: Re: [rt.cpan.org #17539] TimeZoneCatalog.pm does not contain AKST9AKDT
Date: Tue, 21 Feb 2006 14:11:39 -0600 (CST)
To: "ljacobs [...] 10east.com via RT" <bug-DateTime-TimeZone [...] rt.cpan.org>
From: Dave Rolsky <autarch [...] urth.org>
On Tue, 21 Feb 2006, ljacobs@10east.com via RT wrote: Show quoted text
> Thanks for letting me know. Since it appears that this was a Debian > patch that was not passed along to you, I'd like to let you know about > one other difference that we found between Debian's and Gentoo's version > of TimeZoneCatalog.pm (based on version 0.38): > > --- TimeZoneCatalog.pm-0.38 2006-02-07 17:55:08.000000000 -0500 > +++ TimeZoneCatalog.pm-Debian 2006-02-21 09:41:31.000000000 -0500 > @@ -908,5 +908,5 @@ > 'US/Arizona' => 'America/Phoenix', > 'US/Central' => 'America/Chicago', > - 'US/East-Indiana' => 'America/Indiana/Indianapolis', > + 'US/East-Indiana' => 'America/Indianapolis', > 'US/Eastern' => 'America/New_York', > 'US/Hawaii' => 'Pacific/Honolulu', > > I did not mention that one before as I am not sure what all the > implications of that change are, or why they might have done it, but > there it is.
Hmm, that link comes directly from the Olson data, so I'm not going to change it. I have no idea why they're doing that. IIRC, America/Indianapolis is itself a link to America/Indiana/Indianapolis, so for now that patch is basically a no-op. -dave /*=================================================== VegGuide.Org www.BookIRead.com Your guide to all that's veg. My book blog ===================================================*/
  Sun May 21 13:44:45 2006 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'
  Sun May 21 13:44:45 2006 DROLSKY [...] cpan.org - Fixed in 0.43 added