THROWAWAY: Don't load unmodified generated external classes dont-load-generated-external
authorDagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Thu, 17 Sep 2015 22:12:51 +0000 (23:12 +0100)
committerDagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Thu, 17 Sep 2015 22:24:28 +0000 (23:24 +0100)
commitbac32da1dfa1d3cf12ba439a210421f722ce43dd
tree0e93a31bee570aaffcdd5e39e760592637f154cd
parentc2ecce69c18f38fd94402e3ad235fbc25a0a8a00
THROWAWAY: Don't load unmodified generated external classes

Loading external classes is only useful in dynamic mode, or when
transitioning to static mode, and loading code we generated ourselves is
never useful.

However, mst pointed out a better approach on IRC, so this commit is
only so I can pick it up from elsewhere and change it to this approach:

> I would argue that if the installed thing has a Schema::Loader md5 in
> it, you should ignore it, and you should only bring custom code in if
> you're writing new files.

> That should handle the dynamic->static transition case, and then you can
> just output a warning or something if you see an extra file but ignore
> it, with a note of "turn option X on if you really wanted this".
lib/DBIx/Class/Schema/Loader/Base.pm
t/23dumpmore.t
t/lib/dbixcsl_dumper_tests.pm