Fix all the tests to get it right, use Cwd rather than FindBin

[catagits/Catalyst-Runtime.git] / lib / Catalyst / Utils.pm

diff --git a/lib/Catalyst/Utils.pm b/lib/Catalyst/Utils.pm
index 7bdbca0..306d1b8 100644 (file)
--- a/lib/Catalyst/Utils.pm
+++ b/lib/Catalyst/Utils.pm
@@ -6,10 +6,10 @@ use HTTP::Request;
 use Path::Class;
 use URI;
 use Carp qw/croak/;
-use Cwd;
 use Class::MOP;
 use String::RewritePrefix;
 use List::MoreUtils qw/ any /;
+use Cwd qw/ cwd /;
 
 use namespace::clean;
 
@@ -162,7 +162,7 @@ Returns a list of files which can be tested to check if you're inside a checkout
 =cut
 
 sub dist_indicator_file_list {
-    qw/ Makefile.PL Build.PL dist.init /;
+    qw/ Makefile.PL Build.PL dist.ini /;
 }
 
 =head2 home($class)
@@ -207,8 +207,8 @@ sub home {
 
 =head2 find_home_unloaded_in_checkout ($path)
 
-Tries to determine if C<$path> (or the current directory if not supplied)
-looks like a checkout. Any leading lib or blib components
+Tries to determine if C<$path> (or cwd if not supplied)
+looks like a checkout. Any leading lib, script or blib components
 will be removed, then the directory produced will be checked
 for the existence of a C<< dist_indicator_file_list() >>.
 
@@ -220,25 +220,26 @@ sub find_home_unloaded_in_checkout {
     my ($path) = @_;
     $path ||= cwd() if !defined $path || !length $path;
     my $home = dir($path)->absolute->cleanup;
-
     # pop off /lib and /blib if they're there
-    $home = $home->parent while $home =~ /b?lib$/;
-
-    # only return the dir if it has a Makefile.PL or Build.PL or dist.ini
-    if (any { $_ } map { -f $home->file($_) } dist_indicator_file_list()) {
-
-        # clean up relative path:
-        # MyApp/script/.. -> MyApp
-
-        my $dir;
-        my @dir_list = $home->dir_list();
-        while (($dir = pop(@dir_list)) && $dir eq '..') {
-            $home = dir($home)->parent->parent;
+    # pop off /script if it's there.
+
+    do {
+        # only return the dir if it has a Makefile.PL or Build.PL or dist.ini
+        if (any { $_ } map { -f $home->file($_) } dist_indicator_file_list()) {
+            # clean up relative path:
+            # MyApp/script/.. -> MyApp
+
+            my $dir;
+            my @dir_list = $home->dir_list();
+            while (($dir = pop(@dir_list)) && $dir eq '..') {
+                $home = dir($home)->parent->parent;
+            }
+            return $home->stringify;
         }
-
-        return $home->stringify;
+        $home = $home->parent;
     }
-
+    while # pop off /lib and /blib or /script or /t/ if they're there
+        ($home =~ /b?lib$/ || $home =~ /script$/ || $home =~ /\/t(\/|$)/);
 }
 
 =head2 prefix($class, $name);