[Dbix-class] Improving Schema::Loader's dump_overwrite behaviour

[Brandon Black]

On 7/18/06, Matt S Trout <dbix-class at trout.me.uk> wrote:
> Brandon Black wrote:
> > On 7/18/06, Nilson Santos Figueiredo Junior <acid06 at gmail.com> wrote:
> >> Attached you'll find a patch which changes dump_overwrite behaviour so
> >> that when it's false (i.e. it shouldn't overwrite files), instead
> >> throwing an exception when an already existing file is found, it
> >> merely warns and skips it.
> >>
> >> This is rather useful when you're e.g. adding new tables to an
> >> existing Catalyst application and want to use the Helper with
> >> "create=static" for code generation.
> >>
> >> The patch fixes one test case so that it complies with this new
> >> behaviour (thus, Test::Warn was added as a build dependency). It also
> >> changes two or three words on the docs reflecting the change.
> >>
> >
> > Thanks.  Its applied and should show up in the 0.03005 release.
>
> If local $SIG{WARN} would be sufficient to the test please don't add another
> dependency just to get syntactic sugar in code most users will never see.
>

You're right of course, I guess it was too early in the morning for me
to think that clearly, the test has been amended now.

-- Brandon