This morning I was playing about with some i18n and finally got bored if
having to restart the server myself after editing .po files.
I had a look at myapp_server.pl's options and spotted the useful looking
-restartregex option.
So, I edited my startup alias to include:
-rr '.(?:pm|po|yaml|yml)$'
only to find that there were shell (bash) errors being emitted.
It turns out that although I escaped the regex option on the
command-line this isn't being preserved in C::E::HTTP.
I'm not sure how best to test it, but I've attached a proposed patch for
the module.
Chisel
--
Chisel Wright
e: [EMAIL PROTECTED]
w: http://www.herlpacker.co.uk/
If you were MEANT to understand it, we wouldn't have called it 'code'
--- /usr/local/share/perl/5.8.8/Catalyst/Engine/HTTP.pm 2007-12-04 22:36:25.000000000 +0000
+++ /home/chisel/CatEngineHTTP.pm 2007-12-04 09:07:39.000000000 +0000
@@ -332,8 +332,8 @@
### are available again for the exec'ed process --kane
use Config;
$ENV{PERL5LIB} .= join $Config{path_sep}, @INC;
-
- exec $^X . ' "' . $0 . '" ' . join( ' ', @{ $options->{argv} } );
+
+ exec $^X . ' "' . $0 . '" ' . join( ' ', map(qq{'$_'},@{ $options->{argv} }) );
}
exit;
_______________________________________________
List: [email protected]
Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
Searchable archive: http://www.mail-archive.com/[EMAIL PROTECTED]/
Dev site: http://dev.catalyst.perl.org/
