Skip Menu |

This queue is for tickets about the Time-dt CPAN distribution.

Report information
The Basics
Id: 122111
Status: open
Priority: 0/
Queue: Time-dt

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc: ESAYM [...] cpan.org
AdminCc:

Bug Information
Severity: (no value)
Broken in: v0.0.1
Fixed in: (no value)



CC: ESAYM [...] cpan.org
Subject: Not enough arguments for Time::Piece::_strptime
With the latest development version of Time::Piece (1.31_03) the load test fails: # Failed test 'use Time::dt;' # at t/00-load.t line 8. # Tried to use 'Time::dt'. # Error: Not enough arguments for Time::Piece::_strptime at /tmpfs/.cpan-build-cpansand/2017061600/Time-dt-v0.0.1-0/blib/lib/Time/dt.pm line 123, near "$format)" # Compilation failed in require at t/00-load.t line 8. # BEGIN failed--compilation aborted at t/00-load.t line 8. FAILED--Further testing stopped: cannot load Time::dt I did not look further into the issue, but it looks like a private function is used here.
On Fri Jun 16 12:27:38 2017, SREZIC wrote: Show quoted text
> With the latest development version of Time::Piece (1.31_03) the load > test fails: > > # Failed test 'use Time::dt;' > # at t/00-load.t line 8. > # Tried to use 'Time::dt'. > # Error: Not enough arguments for Time::Piece::_strptime at > /tmpfs/.cpan-build-cpansand/2017061600/Time-dt-v0.0.1- > 0/blib/lib/Time/dt.pm line 123, near "$format)" > # Compilation failed in require at t/00-load.t line 8. > # BEGIN failed--compilation aborted at t/00-load.t line 8. > FAILED--Further testing stopped: cannot load Time::dt > > > I did not look further into the issue, but it looks like a private > function is used here.
The function signature for the (private) method Time::Piece::_strptime has changed. Really shouldn't be reaching into a private space like that but... can be fixed by first initializing the global TP default locale and then passing that as a function arg. Something like: &Time::Piece::_default_locale(); #need only call once then: Time::Piece->_strptime( "time string", "format", $is_local, &Time::Piece::_locale() );