Ensure deps can run after mix deps.get

Author: José Valim

lib/mix/lib/mix/deps/loader.ex

@@ -124,11 +124,7 @@ defmodule Mix.Deps.Loader do
 
   defp scm_status(scm, opts) do
     if scm.checked_out?(opts) do
-      if File.exists?(Path.join(opts[:build], ".compile")) do
-        :compile
-      else
-        { :ok, nil }
-      end
+      { :ok, nil }
     else
       { :unavailable, opts[:dest] }
     end
@@ -203,13 +199,19 @@ defmodule Mix.Deps.Loader do
 
   defp validate_app(Mix.Dep[opts: opts, requirement: req, app: app, status: status] = dep) do
     opts_app = opts[:app]
-
-    if not ok?(status) or opts_app == false do
-      dep
-    else
-      path = if is_binary(opts_app), do: opts_app, else: "ebin/#{app}.app"
-      path = Path.expand(path, opts[:build])
-      dep.status app_status(path, app, req)
+    build    = opts[:build]
+
+    cond do
+      not ok?(status) ->
+        dep
+      File.exists?(Path.join(opts[:build], ".compile")) ->
+        dep.status(:compile)
+      opts_app == false ->
+        dep
+      true ->
+        path  = if is_binary(opts_app), do: opts_app, else: "ebin/#{app}.app"
+        path  = Path.expand(path, build)
+        dep.status app_status(path, app, req)
     end
   end
 

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

@@ -25,6 +25,10 @@ defmodule Mix.Tasks.DepsPathTest do
       Mix.Tasks.Deps.Get.run ["--all"]
       assert File.exists?("_build/dev/lib/raw_repo/.compile")
       assert File.exists?("_build/test/lib/raw_repo/.compile")
+
+      Mix.Tasks.Run.run ["-e", "Mix.shell.info RawRepo.hello"]
+      assert_received { :mix_shell, :info, ["* Compiling raw_repo"] }
+      assert_received { :mix_shell, :info, ["world"] }
     end
   end