Bug #103271 for Log-Dispatch-FileRotate: WARNING: the TZ Date::Manip config variable is deprecated

Thu Apr 02 09:32:10 2015 ribugent [...] gmail.com - Ticket created

Subject:

WARNING: the TZ Date::Manip config variable is deprecated

Hello, i've detected this warning on my company after upgrading Date::Manip to 6.49: # There were 1 warning(s) # Previous test 0 '' # WARNING: the TZ Date::Manip config variable is deprecated # and will be removed in March 2016. Please use # the SetDate or ForceDate config variables instead. # at /usr/local/share/perl/5.20.2/Date/Manip/TZ_Base.pm line 37. # Date::Manip::TZ_Base::_config_var(Date::Manip::TZ=HASH(0xa5a94b0), "TZ", "CET") called at /usr/local/share/perl/5.20.2/Date/Manip/Obj.pm line 250 # Date::Manip::Obj::config(Date::Manip::Date=HASH(0xa5a9498), "TZ", "CET") called at /usr/local/share/perl/5.20.2/Date/Manip/DM6.pm line 96 # Date::Manip::DM6::Date_Init("TZ=CET") called at /usr/local/share/perl/5.20.2/Log/Dispatch/FileRotate.pm line 68 # Log::Dispatch::FileRotate::new("Log::Dispatch::FileRotate", "min_level", "debug", "filename", "/tmp/test.log", "l4p_depends_on", ARRAY(0xaa9c7d8), "TZ", "CET", ...) called at /usr/local/share/perl/5.20.2/Log/Log4perl/Appender.pm line 81 Please consider release a fix before March 2016. Thanks in advance. Gerard Ribugent

Wed Aug 05 12:07:00 2015 mcarey [...] ucar.edu - Correspondence added

Subject:	[rt.cpan.org #103271]
Date:	Wed, 5 Aug 2015 10:06:49 -0600
To:	bug-Log-Dispatch-FileRotate [...] rt.cpan.org
From:	Maxwell Carey <mcarey [...] ucar.edu>

Here's a patch to replace the deprecated TZ variable with SetDate as described in the docs (https://metacpan.org/pod/Date::Manip::Config#TZ). SetDate is not compatible with Date::Manip version 5, so this change will break for Perl versions before 5.10. Time zone handling in Date::Manip v5 is supposedly pretty bad anyway, so that might not be a big problem.

Message body is not shown because sender requested not to inline it.

Wed Aug 05 12:07:01 2015 The RT System itself - Status changed from 'new' to 'open'

Mon May 23 19:09:20 2016 MRHORNER [...] cpan.org - Correspondence added

When is the TZ patch going to make it out on CPAN? # WARNING: the TZ Date::Manip config variable is deprecated # and will be removed in March 2016. Please use # the SetDate or ForceDate config variables instead. We are almost 2 months past the date of deprecation for TZ Date::Manip. I would really like to simply update my module from CPAN vs. manually patching each machine. Thanks very much, --dave https://dave.thehorners.com/tech-talk/programming/94-perl-programming

Tue Aug 02 15:18:14 2016 MSCHOUT [...] cpan.org - Correspondence added

+1 on this for me. I note that this module has not been updated since 2008. IF you are uninterested in maintaining this module any longer, would you consider at least adding me as COMAINT in pause so that this issue can be fixed? Alternatively, consider putting this module up for adoption via either ADOPTME or HANDOFF permissions (see http://neilb.org/2013/08/07/adoptme.html) -- Regards, Michael Schout

Thu Aug 04 21:25:00 2016 markpf [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #103271] WARNING: the TZ Date::Manip config variable is deprecated
Date:	Fri, 5 Aug 2016 11:24:10 +1000
To:	bug-Log-Dispatch-FileRotate [...] rt.cpan.org
From:	Mark Pfeiffer <markpf [...] gmail.com>

Added MSCHOUT to co-maintainers of Log::Dispatch::FileRotate My apologies I have had heart surgery and also managed to snag a manager's role at Google which means I'm somewhat distracted. On 3 August 2016 at 05:18, MSCHOUT via RT < bug-Log-Dispatch-FileRotate@rt.cpan.org> wrote: Show quoted text

> Queue: Log-Dispatch-FileRotate > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=103271 > > > +1 on this for me. > > I note that this module has not been updated since 2008. IF you are > uninterested in maintaining this module any longer, would you consider at > least adding me as COMAINT in pause so that this issue can be fixed? > > Alternatively, consider putting this module up for adoption via either > ADOPTME or HANDOFF permissions (see http://neilb.org/2013/08/07/ > adoptme.html) > -- > Regards, > Michael Schout >

-- cheers markpf@gmail.com

Sat Oct 01 15:36:55 2016 MSCHOUT [...] cpan.org - Correspondence added

Appologies for not taking care of this sooner. I hadn't realized I was granted COMAINT yet. I have just uploaded v1.20 to PAUSE which fixes this issue. I ened up using version.pm to check if an older version of Date::Manip is installed and we use TZ in that case in order to maintain backwards compat with older perls. This should show up in CPAN shortly. Thanks for your patience -- Regards, Michael Schout

Sat Oct 01 15:36:56 2016 MSCHOUT [...] cpan.org - Status changed from 'open' to 'patched'

Tue Feb 07 12:18:23 2017 MSCHOUT [...] cpan.org - Fixed in 1.20 added

Thu Jul 27 14:53:52 2017 MSCHOUT [...] cpan.org - Status changed from 'patched' to 'resolved'