Skip Menu |

This queue is for tickets about the DBIx-Class-Cursor-Cached CPAN distribution.

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

History Show all quoted text
  Mon Oct 18 18:19:02 2010 TODDR [...] cpan.org - Ticket created
Subject: Missing module dependency DBD::SQLite
This module fails it's test suite without DBD::SQLite. SQLite is not mentioned as a dependency in the Makefile.PL or META.yml files.
  Mon Oct 18 18:25:30 2010 ash_rtcpan [...] firemirror.com - Correspondence added
It does how ever have a dependency on DBIx::Class which in turn has a build_requires on DBD::SQLite. I've rejected this bug - feel free to re-open it with more info if you think I've made a mistake.
  Mon Oct 18 18:25:31 2010 The RT System itself - Status changed from 'new' to 'open'
  Mon Oct 18 18:25:31 2010 ash_rtcpan [...] firemirror.com - Status changed from 'open' to 'rejected'
  Wed Oct 20 01:58:30 2010 TODDR [...] cpan.org - Correspondence added
On Mon Oct 18 18:25:30 2010, ASH wrote: Show quoted text
> It does how ever have a dependency on DBIx::Class which in turn has a > build_requires on > DBD::SQLite. > > I've rejected this bug - feel free to re-open it with more info if you > think I've made a mistake.
build_requires does not imply that the module is required after build. some installers even give you an option to not install them after they're used. In the case of automating building RPMs, I only need Test::Exception to build DBIC, not install it. So depending on other modules to get your dependencies falls apart for that. Honestly it's my feel that if the module needs it for build or regular use, there's alot of value to explicitly stating that in your makefile/meta
  Wed Oct 20 01:58:31 2010 The RT System itself - Status changed from 'rejected' to 'open'
  Wed Oct 20 06:06:38 2010 ribasushi [...] leporine.io - Correspondence added
On Mon Oct 18 18:19:02 2010, TODDR wrote: Show quoted text
> This module fails it's test suite without DBD::SQLite. SQLite is not > mentioned as a dependency in > the Makefile.PL or META.yml files.
Fixed, http://dev.catalystframework.org/svnweb/bast/revision?rev=9765 Cheers!
  Wed Oct 20 06:06:39 2010 ribasushi [...] leporine.io - Status changed from 'open' to 'patched'
  Tue Jan 25 12:41:14 2011 justin.d.hunter [...] gmail.com - Correspondence added
fixed in 1.001000
  Tue Jan 25 12:41:15 2011 The RT System itself - Status changed from 'patched' to 'open'
  Tue Jan 25 12:41:15 2011 justin.d.hunter [...] gmail.com - Status changed from 'open' to 'resolved'