[Catalyst-commits] r9817 - in trunk/Catalyst-Controller-WrapCGI: . lib/Catalyst/Controller

[caelum at dev.catalyst.perl.org]

Author: caelum
Date: 2009-04-24 05:44:13 +0100 (Fri, 24 Apr 2009)
New Revision: 9817

Modified:
   trunk/Catalyst-Controller-WrapCGI/Changes
   trunk/Catalyst-Controller-WrapCGI/Makefile.PL
   trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/CGIBin.pm
   trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/WrapCGI.pm
Log:
C::C::CGIBin: add support for __DATA__ sections in CGIs

Modified: trunk/Catalyst-Controller-WrapCGI/Changes
===================================================================
--- trunk/Catalyst-Controller-WrapCGI/Changes	2009-04-24 02:14:28 UTC (rev 9816)
+++ trunk/Catalyst-Controller-WrapCGI/Changes	2009-04-24 04:44:13 UTC (rev 9817)
@@ -22,3 +22,6 @@
 
 0.0027  2009-04-03 14:55:34
     Add 'kill_env' and default to killing 'MOD_PERL' from environment.
+
+0.0028  2009-04-24 04:40:39
+    Add support for __DATA__ sections in cgis for C::CGIBin

Modified: trunk/Catalyst-Controller-WrapCGI/Makefile.PL
===================================================================
--- trunk/Catalyst-Controller-WrapCGI/Makefile.PL	2009-04-24 02:14:28 UTC (rev 9816)
+++ trunk/Catalyst-Controller-WrapCGI/Makefile.PL	2009-04-24 04:44:13 UTC (rev 9817)
@@ -6,7 +6,6 @@
 include  'Module::AutoInstall';
 
 requires 'Catalyst' => '5.7007';
-requires 'Class::C3' => '0.20';
 requires 'parent';
 requires 'Task::Weaken';
 requires 'HTTP::Request::AsCGI';
@@ -16,6 +15,13 @@
 requires 'File::Slurp';
 requires 'namespace::clean';
 
+requires 'MRO::Compat';
+
+if($] < 5.009_005) {
+    requires 'Class::C3::XS' => '0.08';
+    requires 'Class::C3' => '0.20';
+}
+
 test_requires 'Catalyst::Plugin::Static::Simple';
 
 build_requires 'Test::More' => '0.86';

Modified: trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/CGIBin.pm
===================================================================
--- trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/CGIBin.pm	2009-04-24 02:14:28 UTC (rev 9816)
+++ trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/CGIBin.pm	2009-04-24 04:44:13 UTC (rev 9817)
@@ -3,7 +3,8 @@
 use strict;
 use warnings;
 
-use Class::C3;
+use MRO::Compat;
+use mro 'c3';
 use File::Slurp 'slurp';
 use File::Find::Rule ();
 use Catalyst::Exception ();
@@ -12,6 +13,7 @@
 use Symbol 'gensym';
 use List::MoreUtils 'any';
 use IO::File ();
+use Carp;
 use namespace::clean -except => 'meta';
 
 use parent 'Catalyst::Controller::WrapCGI';
@@ -22,11 +24,11 @@
 
 =head1 VERSION
 
-Version 0.005
+Version 0.006
 
 =cut
 
-our $VERSION = '0.005';
+our $VERSION = '0.006';
 
 =head1 SYNOPSIS
 
@@ -194,12 +196,25 @@
 sub wrap_perl_cgi {
     my ($self, $cgi, $action_name) = @_;
 
-    do {
+    my $code = slurp $cgi;
+
+    $code =~ s/^__DATA__\r?\n(.*)//ms;
+    my $data = $1;
+
+    my $coderef = do {
         no warnings;
         eval ' 
             package Catalyst::Controller::CGIBin::_CGIs_::'.$action_name.';
-            sub {' . slurp($cgi) . '}'
-    }
+            sub {'
+                . 'local *DATA;'
+                . q{open DATA, '<', \$data;}
+                . $code
+         . '}';
+    };
+
+    croak __PACKAGE__ . ": Could not compile $cgi to coderef: $@" if $@;
+
+    $coderef
 }
 
 =head2 $self->wrap_nonperl_cgi($path, $action_name)

Modified: trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/WrapCGI.pm
===================================================================
--- trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/WrapCGI.pm	2009-04-24 02:14:28 UTC (rev 9816)
+++ trunk/Catalyst-Controller-WrapCGI/lib/Catalyst/Controller/WrapCGI.pm	2009-04-24 04:44:13 UTC (rev 9817)
@@ -15,11 +15,11 @@
 
 =head1 VERSION
 
-Version 0.0027
+Version 0.0028
 
 =cut
 
-our $VERSION = '0.0027';
+our $VERSION = '0.0028';
 
 =head1 SYNOPSIS