Merge pull request #10157 from prathamesh-sonpatki/app_loader

Searching for rails executable correctly

1 files changed, 11 insertions, 16 deletions

diff --git a/railties/lib/rails/app_rails_loader.rb b/railties/lib/rails/app_rails_loader.rb
index 1c64f2dcc6..ef6fcd81b4 100644
--- a/railties/lib/rails/app_rails_loader.rb
+++ b/railties/lib/rails/app_rails_loader.rb
@@ -6,22 +6,22 @@ module Rails
     EXECUTABLES = ['bin/rails', 'script/rails']
 
     def self.exec_app_rails
-      cwd   = Dir.pwd
-
-      exe   = find_executable
-      exe ||= find_executable_in_parent_path
-      return unless exe
+      cwd = Dir.pwd
+      pathname = Pathname.new(Dir.pwd)
+
+      until exe = find_executable
+        # Return to working directory if root is hit without finding executable
+        Dir.chdir(cwd) and return if pathname.root?
+        # Otherwise keep moving upwards in search of executable
+        Dir.chdir("..")
+        pathname = pathname.parent
+      end
 
       contents = File.read(exe)
 
       # This is the Rails executable, let's use it
       if contents =~ /(APP|ENGINE)_PATH/
-        exec RUBY, exe, *ARGV if find_executable
-        Dir.chdir("..") do
-          # Recurse in a chdir block: if the search fails we want to be sure
-          # the application is generated in the original working directory.
-          exec_app_rails unless cwd == Dir.pwd
-        end
+        exec RUBY, exe, *ARGV
 
       # This is a Bundler binstub. Stop and explain how to upgrade.
       elsif exe =~ /bin\/rails$/ && contents =~ /This file was generated by Bundler/
@@ -59,10 +59,5 @@ generate it and add it to source control:
       EXECUTABLES.find { |exe| File.exists?(exe) }
     end
 
-    def self.find_executable_in_parent_path(path = Pathname.new(Dir.pwd).parent)
-      EXECUTABLES.find do |exe|
-        File.exists?(exe) || !path.root? && find_executable_in_parent_path(path.parent)
-      end
-    end
   end
 end