Mailing List Archive

On 25 Jun 2009, at 01:42, Stuart Watt wrote:
> Here is a proposed patch to Catalyst::Plugin::ConfigLoader that
> allows it to use environment variables as a built-in macro for
> configuration. I use this technique a lot in managing multiple
> FastCGI configurations.
>
> This is the first time I've mailed a patch, so any feedback or
> comments will be welcome. I did read the guidelines, and I hope the
> accompanying tests are appropriate and adequate.

Looks great to me, useful feature - thanks!

Applied with attribution as r10698 (http://dev.catalyst.perl.org/
svnweb/Catalyst/revision/?rev=10698).

Cheers
t0m


_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev

Documentation also should be fixed - now we have more than three
default macros ;)

--- /usr/local/share/perl/5.10.0/Catalyst/Plugin/ConfigLoader.pm
2009-08-13 12:46:41.000000000 +0000
+++ /tmp/ConfigLoader.pm 2009-08-18 22:52:54.006574352 +0000
@@ -273,7 +273,7 @@

=head2 config_substitutions( $value )

-This method substitutes macros found with calls to a function. There are three
+This method substitutes macros found with calls to a function. There are some
default macros:

=over 4

--
Sincerely yours,
Oleg Kostyuk (CUB-UANIC)

_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev

On Tue, Aug 18, 2009 at 7:56 PM, Oleg Kostyuk<cub.uanic@gmail.com> wrote:
> Documentation also should be fixed - now we have more than three
> default macros ;)

A slightly different wording committed as svn r11163.

Thanks.

--
Brian Cassidy (brian.cassidy@gmail.com)

_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev