Re: DateTime::Format::Strptime fails test

2009-07-13 Thread Dave Rolsky
On Mon, 13 Jul 2009, Rick Measham wrote: I believe there's an error in DateTime::Locale::Base that is screwing with my fix. Line 277 turns the CLDR notation 'y' into the strftime notation '%y'. From my reading of http://www.unicode.org/reports/tr35/tr35-9.html#Date_Format_Patterns 'y' should

RE: DateTime::Format::Strptime fails test

2009-07-13 Thread Metz, Bobby
Dave, I'm not sure I understood your last post. Did you mean that those strftime methods only found in DateTime::Locale::Base are going away? Or did you mean that strftime methods in DateTime proper are being deprecated and removed? If yes to the later, is DateTime::Format::Strptime

RE: DateTime::Format::Strptime fails test

2009-07-13 Thread Metz, Bobby
All, I'm sorry all to reply to my own post. I should have read the original message first before replying to Dave's message. I see that the module I'm questioning is at the root of the issue, so my apologies again. As someone who uses DateTime::Format::Strptime quite a bit,

RE: DateTime::Format::Strptime fails test

2009-07-13 Thread Dave Rolsky
On Mon, 13 Jul 2009, Metz, Bobby wrote: I'm sorry all to reply to my own post. I should have read the original message first before replying to Dave's message. I see that the module I'm questioning is at the root of the issue, so my apologies again. As someone who uses

Re: DateTime::Format::Strptime fails test

2009-07-13 Thread Rick Measham
Dave Rolsky wrote: All those methods that convert to strftime patterns are deprecated and will go away in a future release, so even if I fixed this bug, if you're relying on them, your code will break eventually. Erm .. I'm confused. DateTime::Locale will no longer provide the strftime

[ANNOUNCE] DateTime::Format::Strptime 1.1000

2009-07-13 Thread Rick Measham
This release allows the 'pattern' to be a regex. This is handy for situations like: pattern = '%Y-%m-%d[T ]%H:%M:%S' where you'll accept either the 'T' or space delimiter in ISO8601 type datetimes. This release depends on DateTime::Locale 0.43 and the locale tests expect the data provided

Item #1 returned by STORABLE_freeze for DateTime is not a reference

2009-07-13 Thread Bill Moseley
Does Item #1 mean the first list element returned by STORABLE_freeze? return $serialized, $self-{locale}, $self-{tz}, \$self-{formatter}; Because, indeed it isn't a reference, but a scalar. I have some code that is serializing items that have DateTime objects (of course) and I'm seeing

Re: [ANNOUNCE] DateTime::Format::Strptime 1.1000

2009-07-13 Thread Dave Rolsky
On Tue, 14 Jul 2009, Rick Measham wrote: This release depends on DateTime::Locale 0.43 and the locale tests expect the data provided by that module. This isn't future-proof, but Dave says that the methods that provide the %x, %X and %c patterns to strftime are deprecated. Once the target

Re: Item #1 returned by STORABLE_freeze for DateTime is not a reference

2009-07-13 Thread Yitzchak Scott-Thoennes
On Mon, July 13, 2009 6:13 pm, Bill Moseley wrote: (in cleanup) Item #1 returned by STORABLE_freeze for DateTime is not a reference at ../../lib/Storable.pm (autosplit into ../../lib/auto/Storable/_freeze.al) line 339 during global destruction, at line 327 It's expected that this

Re: [ANNOUNCE] DateTime::Format::Strptime 1.1000

2009-07-13 Thread Shane McCarron
Dave Rolsky wrote: As far the locale data, that target will never stop moving. I'll keep releasing new versions as the CLDR folks update their data. and I, for one, thank you for it. Tea is on me the next time we manage to be in the same space at the same time. -- Shane P. McCarron