Skip Menu |

This queue is for tickets about the DBIx-Class-DateTime-Epoch CPAN distribution.

Report information
The Basics
Id: 74576
Status: resolved
Priority: 0/
Queue: DBIx-Class-DateTime-Epoch
People
Owner: Nobody in particular
Requestors: TODDR [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.08
Fixed in: (no value)

History Show all quoted text
  Tue Jan 31 18:20:54 2012 TODDR [...] cpan.org - Ticket created
Subject: Missing Dependency - DateTime::Format::SQLite
It looks like DateTime::Format::SQLite needs to be added as a require. This module will not work without it. Thanks! Todd
  Tue Jan 31 18:24:22 2012 TODDR [...] cpan.org - Correspondence added
oh also: META.yml implies that EU::MM 6.62 is required to build. At the least, I can build with 6.59. That may be a Module::Install bug though... build_requires: ExtUtils::MakeMaker: 6.62 configure_requires: ExtUtils::MakeMaker: 6.62
  Tue Jan 31 18:24:23 2012 TODDR [...] cpan.org - Status changed from 'new' to 'open'
  Tue Jan 31 21:51:01 2012 bricas [...] cpan.org - Correspondence added
On Tue Jan 31 19:20:54 2012, TODDR wrote: Show quoted text
> It looks like DateTime::Format::SQLite needs to be added as a require. > This module will not > work without it.
I think this is somewhat misleading. I believe (I will try to fully confirm this) that, post-installation, a user will be required to have DateTime::Format::$SOMETHING depending on their database of choice. For example, PostgreSQL users will need DateTime::Format::Pg. Or least the documentation for DBIx::Class::InflateColumn::DateTime says so: https://metacpan.org/module/DBIx::Class::InflateColumn::DateTime#DESCRIP TION However, in the test phase, DBICx::TestDatabase uses DBD::SQLite, so we should at least test_requires DateTime::Format::SQLite. A number of people would get that dependency thanks to DBIx::Class::TimeStamp which uses it in its own test phase. Does this make sense?
  Tue Jan 31 21:52:19 2012 bricas [...] cpan.org - Correspondence added
On Tue Jan 31 19:24:22 2012, TODDR wrote: Show quoted text
> oh also: META.yml implies that EU::MM 6.62 is required to build. At > the least, I can build with > 6.59. That may be a Module::Install bug though... > > build_requires: > ExtUtils::MakeMaker: 6.62 > configure_requires: > ExtUtils::MakeMaker: 6.62
This is a consequence of using Module::Install -- not a bug, just a fact, I'd say.
  Wed Feb 01 10:18:54 2012 bricas [...] cpan.org - Correspondence added
On Tue Jan 31 22:52:19 2012, BRICAS wrote: Show quoted text
> On Tue Jan 31 19:24:22 2012, TODDR wrote:
> > oh also: META.yml implies that EU::MM 6.62 is required to build. At > > the least, I can build with > > 6.59. That may be a Module::Install bug though... > > > > build_requires: > > ExtUtils::MakeMaker: 6.62 > > configure_requires: > > ExtUtils::MakeMaker: 6.62
> > This is a consequence of using Module::Install -- not a bug, just a
fact, Show quoted text
> I'd say.
I should clarify that what Module::Install does is require the version of ExtUtils::MakeMaker that is installed on my system. This is a indeed a bug/problem upstream -- but not critical.
  Wed Feb 01 10:20:11 2012 bricas [...] cpan.org - Correspondence added
I've pushed a new version of the module with the following changes: 0.09 2012-02-01 - added a test dependency on DateTime::Format::SQLite (RT #74576) - added some documentation about the installation requirement of an appropriate DateTime::Format::* module for your RDBMS of choice Hope that helps!
  Wed Feb 01 10:20:12 2012 bricas [...] cpan.org - Status changed from 'open' to 'resolved'