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: 25348
Status: resolved
Priority: 0/
Queue: DateTime-TimeZone
People
Owner: Nobody in particular
Requestors: bcbailey [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.62
Fixed in: 0.63

History Show all quoted text
  Thu Mar 08 18:21:26 2007 bcbailey [...] cpan.org - Ticket created
Subject: patch: DateTime::TimeZone::Local::Unix _Readlink bug
I was getting the "Cannot determine time zone" error on CentOS 4 and I tracked it down to a bug in _Readlink. _Readlink is now being called as a class method, so $_[0] is no longer the file name to check. Regards, Bradley C Bailey
Subject: datetime-timezone-local-unix-readlink.patch
--- Unix.pm.orig 2007-03-08 15:58:12.000000000 -0700 +++ Unix.pm 2007-03-08 15:58:21.000000000 -0700 @@ -60,7 +60,7 @@ } } -sub _Readlink { readlink $_[0] } +sub _Readlink { readlink $_[1] } # for systems where /etc/localtime is a copy of a zoneinfo file sub _FindMatchingZoneinfoFile
  Sun Mar 18 22:48:05 2007 chris+rt [...] chrisdolan.net - Correspondence added
From: CDOLAN [...] cpan.org
On Thu Mar 08 18:21:26 2007, bcbailey wrote: Show quoted text
> I was getting the "Cannot determine time zone" error on CentOS 4 and I > tracked it down to a bug in _Readlink. > > _Readlink is now being called as a class method, so $_[0] is no longer > the file name to check. > > Regards, > Bradley C Bailey
I encountered the same bug and your fix worked beautifully Bradley. Thanks much! Dave Rolsky: this is an obviously correct fix. Please apply! Thanks, Chris Dolan
  Sun Mar 18 22:48:06 2007 The RT System itself - Status changed from 'new' to 'open'
  Fri Mar 30 10:52:58 2007 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri Mar 30 10:52:59 2007 DROLSKY [...] cpan.org - Fixed in 0.63 added