Use System.no_halt/1 instead of Process.sleep/1 in mix run

Otherwise :init.get_status will stay in
{:starting, :started} forever.

Author: José Valim

lib/elixir/lib/kernel/cli.ex

@@ -5,7 +5,7 @@ defmodule Kernel.CLI do
     commands: [],
     output: ".",
     compile: [],
-    halt: true,
+    no_halt: false,
     compiler_options: [],
     errors: [],
     pa: [],
@@ -21,6 +21,7 @@ defmodule Kernel.CLI do
 
     {config, argv} = parse_argv(argv)
     System.argv(argv)
+    System.no_halt(config.no_halt)
 
     fun = fn _ ->
       errors = process_commands(config)
@@ -31,7 +32,7 @@ defmodule Kernel.CLI do
       end
     end
 
-    run(fun, config.halt)
+    run(fun)
   end
 
   @doc """
@@ -42,10 +43,10 @@ defmodule Kernel.CLI do
   This function is used by Elixir's CLI and also
   by escripts generated by Elixir.
   """
-  def run(fun, halt \\ true) do
+  def run(fun) do
     {ok_or_shutdown, status} = exec_fun(fun, {:ok, 0})
 
-    if ok_or_shutdown == :shutdown or halt do
+    if ok_or_shutdown == :shutdown or not System.no_halt() do
       {_, status} = at_exit({ok_or_shutdown, status})
 
       # Ensure Logger messages are flushed before halting
@@ -240,7 +241,7 @@ defmodule Kernel.CLI do
   end
 
   defp parse_shared(["--no-halt" | t], config) do
-    parse_shared(t, %{config | halt: false})
+    parse_shared(t, %{config | no_halt: true})
   end
 
   defp parse_shared(["-e", h | t], config) do

lib/elixir/lib/system.ex

@@ -255,6 +255,22 @@ defmodule System do
     :elixir_config.put(:argv, args)
   end
 
+  @doc """
+  Marks if the system should halt or not at the end of ARGV processing.
+  """
+  @spec no_halt(boolean) :: :ok
+  def no_halt(boolean) when is_boolean(boolean) do
+    :elixir_config.put(:no_halt, boolean)
+  end
+
+  @doc """
+  Checks if the system will halt or not at the end of ARGV processing.
+  """
+  @spec no_halt() :: boolean
+  def no_halt() do
+    :elixir_config.get(:no_halt)
+  end
+
   @doc """
   Current working directory.
 

lib/elixir/src/elixir.erl

@@ -63,7 +63,8 @@ start(_Type, _Args) ->
     {bootstrap, false},
     {compiler_options, CompilerOpts},
     {home, unicode:characters_to_binary(Home, Encoding, Encoding)},
-    {identifier_tokenizer, Tokenizer}
+    {identifier_tokenizer, Tokenizer},
+    {no_halt, false}
     | URIConfig
   ],
 

lib/mix/lib/mix/tasks/escript.build.ex

@@ -382,7 +382,6 @@ defmodule Mix.Tasks.Escript.Build do
           ]
 
           io_error(error_message)
-
           :erlang.halt(1)
       end
 
@@ -391,7 +390,7 @@ defmodule Mix.Tasks.Escript.Build do
           load_config(@config)
           start_app(@app)
           args = Enum.map(args, &List.to_string(&1))
-          Kernel.CLI.run(fn _ -> @module.main(args) end, true)
+          Kernel.CLI.run(fn _ -> @module.main(args) end)
 
         error ->
           io_error(["Failed to start Elixir.\n", :io_lib.format('error: ~p~n', [error])])

lib/mix/lib/mix/tasks/run.ex

@@ -84,7 +84,7 @@ defmodule Mix.Tasks.Run do
       )
 
     run(args, opts, head, &Code.eval_string/1, &Code.require_file/1)
-    unless Keyword.get(opts, :halt, true), do: Process.sleep(:infinity)
+    unless Keyword.get(opts, :halt, true), do: System.no_halt(true)
     Mix.Task.reenable("run")
     :ok
   end