Do no start listeners if --no-deps-check is given

Author: josevalim

lib/mix/lib/mix/tasks/compile.ex

@@ -61,6 +61,8 @@ defmodule Mix.Tasks.Compile do
     * `--erl-config` - path to an Erlang term file that will be loaded as Mix config
     * `--force` - forces compilation
     * `--list` - lists all enabled compilers
+    * `--listeners` - starts Mix listeners (they are started by default,
+      unless `--no-listeners` or `--no-deps-check` are given)
     * `--no-app-loading` - does not load .app resource file after compilation
     * `--no-archives-check` - skips checking of archives
     * `--no-compile` - does not actually compile, only loads code and perform checks

lib/mix/lib/mix/tasks/deps.loadpaths.ex

@@ -75,9 +75,11 @@ defmodule Mix.Tasks.Deps.Loadpaths do
 
     Code.prepend_paths(Enum.flat_map(all, &Mix.Dep.load_paths/1), cache: true)
 
-    # For now we only allow listeners defined in dependencies, so
-    # we start them right after adding adding deps to the path
-    if "--no-listeners" not in args do
+    # For now we only allow listeners defined in dependencies,
+    # so we start them right after adding adding deps to the path,
+    # as long as we are sure they have been compiled
+    if "--listeners" in args or
+         ("--no-listeners" not in args and "--no-deps-check" not in args) do
       Mix.PubSub.start_listeners()
     end
 

lib/mix/lib/mix/tasks/loadpaths.ex

@@ -21,6 +21,8 @@ defmodule Mix.Tasks.Loadpaths do
 
   ## Command line options
 
+    * `--listeners` - starts Mix listeners (they are started by default,
+      unless `--no-listeners` or `--no-deps-check` are given)
     * `--no-archives-check` - does not check archives
     * `--no-compile` - does not compile dependencies, only check and load them
     * `--no-deps-check` - does not check dependencies, only load available ones

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

@@ -49,6 +49,8 @@ defmodule Mix.Tasks.Run do
   ## Command-line options
 
     * `--eval`, `-e` - evaluates the given code
+    * `--listeners` - starts Mix listeners (they are started by default,
+      unless `--no-listeners` or `--no-deps-check` are given)
     * `--require`, `-r` - executes the given pattern/file
     * `--parallel`, `-p` - makes all requires parallel
     * `--preload-modules` - preloads all modules defined in applications

lib/mix/test/mix/tasks/compile_test.exs

@@ -425,9 +425,12 @@ defmodule Mix.Tasks.CompileTest do
       File.write!("src/b.erl", "-module(b).")
       File.write!("src/c.erl", "-module(c).")
 
-      # Ensure we can boot with compilation and listeners if desired
+      # Ensure we can boot without compilation and listeners if desired
       assert mix(["loadpaths", "--no-compile", "--no-listeners"]) == ""
 
+      # Ensure we can boot only with --no-deps-check if desired
+      assert mix(["loadpaths", "--no-deps-check"]) == ""
+
       # Now setup dependencies
       mix(["deps.compile"])