[Catalyst] Patch to make ConfigLoader load "local" files

[John Tate]

Sorry. This time with an attachment...

John Tate wrote:
>
> Hi,
>
> I've been trying to get C::P::ConfigLoader (0.18) to load "local" =

> configuration files. From my reading of the documentation, if have a =

> Config::General configuration file called "myapp.conf", I should be =

> able to load another file named something like "myapp_local.conf", but =

> I've not been able to make it work. I can't find any other reports of =

> this problem, so I'm probably misreading the documentation or misusing =

> the plugin...
>
> However, from looking at the code and after adding my own debug calls =

> to C::P::ConfigLoader, I think there might be a problem with the =

> find_files method. When building the list of config files to load, it =

> returns something like "myapp.conf_local" rather than =

> "myapp_local.conf". That file will never get loaded into the =

> configuration because it doesn't have the right file type to be picked =

> up by Config::Any. Attached is a trivial patch that changes the way =

> the method builds the filename, which makes "myapp_local.conf" get =

> loaded as I'd expect.
>
> John.




-- =

 The Wellcome Trust Sanger Institute is operated by Genome Research =

 Limited, a charity registered in England with number 1021457 and a =

 company registered in England with number 2742969, whose registered =

 office is 215 Euston Road, London, NW1 2BE. =

-------------- next part --------------
A non-text attachment was scrubbed...
Name: configloader.patch
Type: text/x-patch
Size: 625 bytes
Desc: not available
Url : http://lists.scsys.co.uk/pipermail/catalyst/attachments/20071120/a92a=
439c/configloader.bin