Replace backticks with call to exec(), the former spawns a new process, and the latter replaces the current process. In the daemon context we care about the process id so that calls to startstopdaemon won't spawn a new daemon.

1 files changed, 3 insertions, 2 deletions

diff --git a/script/runner b/script/runner
index 73b03847d..32a0e6b7e 100755
--- a/script/runner
+++ b/script/runner
@@ -19,12 +19,13 @@ Dir.chdir(alaveteli_dir) do
 
         # Load the runner in a subprocess
         pid = fork do
-            `bundle exec rails runner #{ARGV[1]}`
+            exec("bundle exec rails runner #{ARGV[1]}")
             exit 0
         end
 
         # If the environment variable PIDFILE is present,
         # write the pid of the daemon process to that file.
+
         if ENV.has_key? "PIDFILE"
             File.open(ENV["PIDFILE"], 'w') do |fh|
                 fh.puts pid
@@ -34,6 +35,6 @@ Dir.chdir(alaveteli_dir) do
         Process.detach(pid)
     else
         # Not daemon mode
-        `bundle exec rails runner #{ARGV[1]}`
+        exec("bundle exec rails runner #{ARGV[1]}")
     end
 end