Bug #115868 for Courriel: Courriel::Builder warns with an updated DateTime but an older DateTime::Format::Mail

Mon Jul 04 17:27:37 2016 olaf [...] wundersolutions.com - Ticket created

Subject:

Courriel::Builder warns with an updated DateTime but an older DateTime::Format::Mail

Looks like this happens with DateTime >= 1.32 and DateTime::Format::Mail < 0.403 Failed test 'Unexpected warning: You passed a locale to the set() method. You should use set_locale() instead, as using set() may alter the local time near a DST bounday. at /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl-modules/lib/perl5/DateTime/Format/Mail.pm line 292. ' at /opt/perl5.20.2/lib/5.20.2/Carp.pm line 168. Trace begun at /opt/perl5.20.2/lib/5.20.2/Carp.pm line 168 Carp::carp('You passed a locale to the set() method. You should use set_locale() instead, as using set() may alter the local time near a DST bounday.') called at /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl-modules/lib/perl5/x86_64-linux/DateTime.pm line 1947 DateTime::set(undef, 'locale', 'en_US') called at /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl-modules/lib/perl5/DateTime/Format/Mail.pm line 292 DateTime::Format::Mail::format_datetime('DateTime::Format::Mail', 'DateTime=HASH(0x20b0d8b8)') called at /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl-modules/lib/perl5/Courriel/Builder.pm line 141 Courriel::Builder::_add_required_headers('ARRAY(0xd7c19a0)') called at /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl-modules/lib/perl5/Courriel/Builder.pm line 118 Courriel::Builder::build_email('HASH(0x20a8a538)', 'HASH(0x20afc080)', 'HASH(0x20a83038)', 'HASH(0x20a83278)', 'HASH(0x20c5bb10)') called at /opt/teamcity/agents/tcagent2/work/10ba80e60b5cd173/lib/MM/Email/GeoIPUpdateReminder.pm line 101

Mon Jul 04 20:12:12 2016 DROLSKY [...] cpan.org - Correspondence added

On Mon Jul 04 17:27:37 2016, OALDERS wrote: Show quoted text

> Looks like this happens with DateTime >= 1.32 and > DateTime::Format::Mail < 0.403 > > Failed test 'Unexpected warning: You passed a locale to the set() > method. You should use set_locale() instead, as using set() may alter > the local time near a DST bounday. at > /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl- > modules/lib/perl5/DateTime/Format/Mail.pm line 292. > ' > at /opt/perl5.20.2/lib/5.20.2/Carp.pm line 168. > Trace begun at /opt/perl5.20.2/lib/5.20.2/Carp.pm line 168 > Carp::carp('You passed a locale to the set() method. You should use > set_locale() instead, as using set() may alter the local time near a > DST bounday.') called at > /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl- > modules/lib/perl5/x86_64-linux/DateTime.pm line 1947 > DateTime::set(undef, 'locale', 'en_US') called at > /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl- > modules/lib/perl5/DateTime/Format/Mail.pm line 292 > DateTime::Format::Mail::format_datetime('DateTime::Format::Mail', > 'DateTime=HASH(0x20b0d8b8)') called at > /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl- > modules/lib/perl5/Courriel/Builder.pm line 141 > Courriel::Builder::_add_required_headers('ARRAY(0xd7c19a0)') called at > /opt/teamcity/agents/tcagent2/local/MaxMind/oalders__update-perl- > modules/lib/perl5/Courriel/Builder.pm line 118 > Courriel::Builder::build_email('HASH(0x20a8a538)', 'HASH(0x20afc080)', > 'HASH(0x20a83038)', 'HASH(0x20a83278)', 'HASH(0x20c5bb10)') called at > /opt/teamcity/agents/tcagent2/work/10ba80e60b5cd173/lib/MM/Email/GeoIPUpdateReminder.pm > line 101

When you install the latest DateTime you should get a conflict warning if you have an old DT::F::Mail installed. That said, I don't think cpanm actually _shows_ any of that. Unfortunately nothing actually uses the conflicts metadata in the META.json :(

Mon Jul 04 20:12:12 2016 The RT System itself - Status changed from 'new' to 'open'

Wed Jul 06 17:28:48 2016 ether [...] cpan.org - Correspondence added

On 2016-07-04 17:12:12, DROLSKY wrote: Show quoted text

> When you install the latest DateTime you should get a conflict warning > if you have an old DT::F::Mail installed. That said, I don't think > cpanm actually _shows_ any of that. Unfortunately nothing actually > uses the conflicts metadata in the META.json :(

Ideally cpanm should upgrade DT:F:Mail after it has installed DateTime, but it isn't doing that yet. Even displaying the warning would be nice though :)

Sat Jul 16 12:15:53 2016 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Jul 16 12:15:53 2016 DROLSKY [...] cpan.org - Broken in 0.42 added