Matt S Trout wrote:
On Tue, Apr 22, 2008 at 09:33:02AM -0400, Christopher H. Laco wrote:Matt S Trout wrote: [snip]Good for your tests. My tests however, use DBIx::Class deploy, as documented as public API, and it blows up on a version issue not listed in prereq.This -is- your bug. the DBIx::Class tests themselves run fine without SQL::Translator becauseI do NOT use SQL::Translator directly. I make sure it is loaded, fulfilling the requirements in the deploy POD. DBIx::Class requires a specific version or it dies...a version not listed as a prereq [or in the pod]. Please fix your code, update your pod, or make it a prereq.they provide a DDL file; deploy() only needs SQLT on the *build* side to generate that file originally, which is why it's not marked as a dependency on the DBIx::Class dist itself. Please fix UUIDColumns' tests.It's not a prereq for basic operation, only for build-side usage of deploy. We don't make every DBD::* a prereq either. The correct way to use deploy here is the way DBIx::Class' tests use it - use SQL::Translator on the author side to generate a DDL file and make sure your tests can find that DDL file. You're right that the deploy POD should be updated to reflect this, but the -code- needing fixing is still the UUIDColumns tests.
Sigh. Most of this is me being in stubborn devil-on-the-left-shoulder mode...missing the important part: people can use the dist.
I wonder if there's a decent way to do what gen-schema.pl does during make dist... Oddly enough, the tests for most of the dists I have that use deploy already use a sql file in the event that SQL::T isn't available.
signature.asc
Description: OpenPGP digital signature
_______________________________________________ List: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/dbix-class IRC: irc.perl.org#dbix-class SVN: http://dev.catalyst.perl.org/repos/bast/DBIx-Class/ Searchable Archive: http://www.grokbase.com/group/[EMAIL PROTECTED]
