Ensure we automatically load deps on compile

Author: José Valim

lib/mix/lib/mix/tasks/deps.check.ex

@@ -83,7 +83,7 @@ defmodule Mix.Tasks.Deps.Check do
       to_prune = Enum.reduce(all, paths, &(&2 -- Mix.Dep.load_paths(&1)))
 
       Enum.map(to_prune, fn path ->
-        # path may not be in code path
+        # Path may not be in code path
         _ = Code.delete_path(path)
         File.rm_rf!(path |> Path.dirname)
       end)

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

@@ -29,6 +29,8 @@ defmodule Mix.Tasks.Deps.Compile do
 
   def run(args) do
     Mix.Project.get!
+    Mix.Task.run "deps.loadpaths"
+
     case OptionParser.parse(args) do
       {_, [], _} ->
         compile(Enum.filter(loaded(env: Mix.env), &compilable?/1))

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

@@ -0,0 +1,16 @@
+defmodule Mix.Tasks.Deps.Loadpaths do
+  use Mix.Task
+
+  @moduledoc """
+  Loads the available dependencies paths.
+  """
+
+  def run(_) do
+    config = Mix.Project.config
+    Mix.Project.build_path(config)
+    |> Path.join("lib/*/ebin")
+    |> Path.wildcard
+    |> List.delete(config[:app] && Mix.Project.compile_path(config))
+    |> Enum.each(&Code.prepend_path/1)
+  end
+end

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

@@ -47,16 +47,12 @@ defmodule Mix.Tasks.Loadpaths do
     end
   end
 
-  defp load_deps(config, args) do
+  defp load_deps(_config, args) do
     unless "--no-deps-check" in args do
       Mix.Task.run "deps.check", args
     end
 
-    Mix.Project.build_path(config)
-    |> Path.join("lib/*/ebin")
-    |> Path.wildcard
-    |> List.delete(config[:app] && Mix.Project.compile_path(config))
-    |> Enum.each(&Code.prepend_path/1)
+    Mix.Task.run "deps.loadpaths"
   end
 
   defp load_project(config, _args) do

lib/mix/test/mix/tasks/deps.git_test.exs

@@ -103,6 +103,16 @@ defmodule Mix.Tasks.DepsGitTest do
       Mix.Tasks.Deps.Update.run ["deps_on_git_repo"]
       assert File.exists?("deps/deps_on_git_repo/.fetch")
       assert File.exists?("deps/git_repo/.fetch")
+
+      # Compile git repo but unload it so...
+      Mix.Tasks.Deps.Compile.run ["git_repo"]
+      assert File.exists?("_build/dev/lib/git_repo/ebin")
+      Code.delete_path("_build/dev/lib/git_repo/ebin")
+
+      # Deps on git repo loads it automatically on compile.
+      Mix.Task.reenable "deps.loadpaths"
+      Mix.Tasks.Deps.Compile.run ["deps_on_git_repo"]
+      assert File.exists?("_build/dev/lib/deps_on_git_repo/ebin")
     end
   after
     purge [GitRepo, GitRepo.Mix]

lib/mix/test/test_helper.exs

@@ -203,6 +203,11 @@ unless File.dir?(target) do
   end
   """
 
+  File.write! Path.join(target, "lib/deps_on_git_repo.ex"), """
+  ## Auto-generated fixture
+  GitRepo.hello
+  """
+
   File.cd! target, fn ->
     System.cmd("git", ~w[init])
     System.cmd("git", ~w[config user.email "[email protected]"])