Skip Menu |

This queue is for tickets about the SQL-Abstract CPAN distribution.

Report information
The Basics
Id: 55416
Status: resolved
Priority: 0/
Queue: SQL-Abstract
People
Owner: Nobody in particular
Requestors: jozef [...] kutej.net
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: 1.61
Fixed in: 1.62

History Show all quoted text
  Wed Mar 10 05:25:24 2010 jozef [...] kutej.net - Ticket created
Subject: modules used in SQL::Abstract::Test as requires and not just test_requires
Hi, as the SQL::Abstract::Test gets installed to the system shouldn't the modules used inside be declared as requires? Test::Builder and Test::Deep are just test_requires atm. This breaks DBIx::Class tests (if SQL::Abstract::Test is installed from distribution package). Or should the DBIx::Class declare those as test requires? Cheers, Jozef
  Wed Mar 10 06:47:01 2010 ribasushi [...] leporine.io - Correspondence added
On Wed Mar 10 05:25:24 2010, JKUTEJ wrote: Show quoted text
> Hi, > > as the SQL::Abstract::Test gets installed to the system shouldn't the > modules used inside be declared as requires? Test::Builder and > Test::Deep are just test_requires atm. This breaks DBIx::Class tests (if > SQL::Abstract::Test is installed from distribution package). Or should > the DBIx::Class declare those as test requires?
This is fixed in the upcoming SQLA version (should hit cpan tonight): http://dev.catalystframework.org/svnweb/bast/revision?rev=8823 http://dev.catalystframework.org/svnweb/bast/revision?rev=8956
  Wed Mar 10 06:47:02 2010 The RT System itself - Status changed from 'new' to 'open'
  Wed Mar 10 06:47:02 2010 ribasushi [...] leporine.io - Status changed from 'open' to 'patched'
  Mon Mar 15 21:58:06 2010 ribasushi [...] leporine.io - Fixed in 1.62 added
  Mon Mar 15 21:58:06 2010 ribasushi [...] leporine.io - Status changed from 'patched' to 'resolved'