Merge
Tomas Doran [Thu, 13 Oct 2011 14:53:32 +0000 (15:53 +0100)]
Changes
Makefile.PL
lib/Catalyst/Devel.pm
lib/Catalyst/Helper.pm
share/Makefile.PL.tt
t/render_file_contents.t

diff --git a/Changes b/Changes
index 1fd2457..2657654 100644 (file)
--- a/Changes
+++ b/Changes
@@ -1,9 +1,15 @@
 This file documents the revision history for Perl extension Catalyst-Devel.
 
-1.35 ????
+        - bump Module::Install dep to 1.02
         - Removed stderr hiding from 'make catalyst_par' to display errors 
           from PAR::Packer
 
+1.35 2011-09-05 13:05:00
+        - Stop requiring Starman and MooseX::Daemonize on Win32 as they're
+          optional components in Catalyst::Runtime and won't install
+          on Windows.
+        - Fix test to work on Windows. 
+
 1.34 2011-08-16 09:02:00
         - Bump required version of Catalyst to 5.9
         - Merge psgi support. catalyst.pl will now generate a default
index e3b05be..ce47a68 100644 (file)
@@ -21,19 +21,24 @@ requires 'File::ChangeNotify' => '0.07';
 requires 'File::Copy::Recursive';
 requires 'Path::Class' => '0.09';
 requires 'Template'    => '2.14';
-requires 'MooseX::Daemonize'; # Optional in scripts
-requires 'Starman'; # Optional in scripts
+
+if (!$Module::Install::AUTHOR && $^O ne 'MSWin32') {
+    requires 'MooseX::Daemonize'; # Optional in scripts
+    requires 'Starman'; # Optional in scripts
+}
 
 # The Catalyst applications this module distribution have a Makefile.PL using
 # Module::Install as well as the Module::Install::Catalyst extension included in
 # this distribution. Therefore we really *depend* on Module::Install to be
 # installed, even though we also use it to build this distribution and include
 # it in its inc/ directory for releases.
-requires 'Module::Install' => '0.91';
+requires 'Module::Install' => '1.02';
 
 author_requires 'IPC::Run3';
 author_requires 'Module::Info';
 author_requires 'File::Find::Rule';
+author_requires 'Test::Pod';
+author_requires 'Test::Pod::Coverage';
 
 test_requires 'Test::More' => '0.94';
 test_requires 'Test::Fatal' => '0.003';
@@ -42,13 +47,6 @@ install_share 'share';
 
 author_tests 't/author';
 
-if ( $^O eq 'MSWin32' ) {
-    # Proc::Background needs this on Win32 but doesn't actually
-    # require it, if it's missing it just dies in the Makefile.PL.
-    requires 'Win32::Process' => '0.04';
-    requires 'Proc::Background';
-}
-
 if (!$ENV{CATALYST_DEVEL_NO_510_CHECK}) {
     use Symbol 'gensym';
     use IPC::Open3;
index 6f2a7eb..bfaa906 100644 (file)
@@ -4,7 +4,7 @@ use strict;
 use warnings;
 
 # Change Catalyst::Helper also.
-our $VERSION             = '1.34';
+our $VERSION             = '1.35';
 our $CATALYST_SCRIPT_GEN = 40;
 
 $VERSION = eval $VERSION;
index a5cf5a2..976de96 100644 (file)
@@ -19,7 +19,7 @@ use namespace::autoclean;
 with 'MooseX::Emulate::Class::Accessor::Fast';
 
 # Change Catalyst/Devel.pm also
-our $VERSION = '1.34';
+our $VERSION = '1.35';
 
 my %cache;
 
index ae47abd..5aab86a 100644 (file)
@@ -1,7 +1,7 @@
 [% startperl %]
 # IMPORTANT: if you delete this file your app will not work as
 # expected.  You have been warned.
-use inc::Module::Install;
+use inc::Module::Install 1.02;
 use Module::Install::Catalyst; # Complain loudly if you don't have
                                # Catalyst::Devel installed or haven't said
                                # 'make dist' to create a standalone tarball.
index 45b0253..0dbcb0d 100644 (file)
@@ -20,7 +20,9 @@ ok( $helper->render_file_contents('example1',  $fn,
 ok -r $fn;
 ok -s $fn;
 my $perms = ( stat $fn )[2] & 07777;
-is $perms, 0677;
+unless ($^O eq 'MSWin32') {
+    is $perms, 0677;
+}
 unlink $fn;
 
 done_testing;