[Catalyst-dev] couple of small patches

Sebastian Riedel sri at oook.de
Wed Apr 27 23:21:10 CEST 2005


Am 27.04.2005 um 22:18 schrieb Geoff Richards:

> I get some test failures with current svn trunk of catalyst
> (revision 535).  The patch attached fixes them.
>
>  * Minor POD tweak.
>
>  * 't/01use.t' fails because Catalyst.pm comes up with an undefined
>    value for '$path' and then tries to pass it to Path::Class.  That
>    causes uninitialized value warnings, so I wrapped definedness
>    tests round a few pieces of code, which at least makes the tests
>    happy.  But does it make any sense to carry on if $home isn't
>    defined?
>
>  * When 'use'ing Path::Class I think it would be better to name the
>    functions being imported.  Not only is it more greppable, but it 
> will
>    work with earlier versions (I tried it first with 0.03 which doesn't
>    export those functions by default).

Thx, applied.

--
sebastian




More information about the Catalyst-dev mailing list