[Dbix-class] Re: SQL::Translator conversion from Class::Base to Moo

Peter Rabbitson rabbit+dbic at rabbit.us
Wed Aug 29 03:43:08 GMT 2012

On Wed, Aug 29, 2012 at 01:15:50AM +0100, Dagfinn Ilmari Mannsåker wrote:
> ilmari at ilmari.org (Dagfinn Ilmari Mannsåker) writes:
> > The code passes the test suites of DBIx::Class (with
> > DBICTEST_SQLT_DEPLOY=1), ::DeploymentHandler and ::Migration, but I
> > would like some testing and review by actual users before merging it to
> > trunk.
> Thanks to frew and jnap for reviewing and testing this.  Unless anyone
> objects I'll merge it in the next few days so we can get a dev release
> out for smoking.

A cursory test of non-DBIC based SQLT dependents[1] turned up
Metabase::Backend::SQLite[2] with a bunch of readonly accessors being called
as setters. Now, Metabase::Backend::SQLite fails one test on currently
released SQLT anyway (please bug xdg), but nevertheless this is might be
an indication that you were too eager on applying 'ro'.


[1] https://metacpan.org/requires/distribution/SQL-Translator?sort=[[2,1]]
[2] https://metacpan.org/release/Metabase-Backend-SQL

More information about the DBIx-Class mailing list