Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 84371
Status: resolved
Priority: 0/
Queue: DateTime-Format-Strptime
People
Owner: Nobody in particular
Requestors: ALUCAS [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: 1.53

History Show all quoted text
  Tue Apr 02 08:05:13 2013 ALUCAS [...] cpan.org - Ticket created
Subject: Tests failing (unexpected zero padding)
Hi, I think this module is no longer compatible with DateTime post version 1. This module has been failing for me on both Windows and Linux test boxes in the last few days. I've attached the build log. I think it has something to do with changes in DateTime (>= v1). The test fails because it's now receiving stray zeros. Any ideas? I noticed there have been some changes specifically around zero padding in the latest DateTime releases.
Subject: datetime_strptime_fail.build.log

Message body is not shown because it is too large.

  Tue Apr 02 10:43:03 2013 MSCHOUT [...] cpan.org - Correspondence added
It seems that the zero padding is intentional in DateTime 1.00. I did a bisect on DateTime, and the commit that breaks this test (in Strptime) is: commit 32464cf62dc9d8c1c9282cee33a30248bd3f8aa5 Author: chansen <chansen@cpan.org> Date: Sat Mar 30 23:14:23 2013 +0100 Fixed the %j conversion specification in ->strftime to return the correct value %j should be padded with zeros if the day of the year is < 100, this matches the DateTime documentation and the POSIX specification which says: Replaced by the day of the year as a decimal number [001,366]. [ tm_yday] So it seems that the test should be patched to expect '2003;056 = 2003-02-25' instead of '2003;56 = 2003-02-25', and Strptime dist should require DateTime >= 1.00
  Tue Apr 02 10:43:04 2013 The RT System itself - Status changed from 'new' to 'open'
  Tue Apr 02 10:45:32 2013 autarch [...] urth.org - Correspondence added
Subject: Re: [rt.cpan.org #84371] Tests failing (unexpected zero padding)
Date: Tue, 2 Apr 2013 09:45:13 -0500 (CDT)
To: MSCHOUT via RT <bug-DateTime-Format-Strptime [...] rt.cpan.org>
From: Dave Rolsky <autarch [...] urth.org>
On Tue, 2 Apr 2013, MSCHOUT via RT wrote: Show quoted text
> Queue: DateTime-Format-Strptime > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=84371 > > > It seems that the zero padding is intentional in DateTime 1.00. I did a bisect on DateTime, and the commit that breaks this test (in Strptime) is:
I release a new DT::F::Strptime with a fix. /*============================================================ http://VegGuide.org http://blog.urth.org Your guide to all that's veg House Absolute(ly Pointless) ============================================================*/
  Tue Apr 02 10:47:47 2013 MSCHOUT [...] cpan.org - Correspondence added
I see you already fixed for v1.53 :)
  Wed Apr 03 00:04:20 2013 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'
  Wed Apr 03 00:04:21 2013 DROLSKY [...] cpan.org - Fixed in 1.53 added