Include archive.check inside deps.check

Author: José Valim

lib/mix/lib/mix/tasks/clean.ex

@@ -47,7 +47,7 @@ defmodule Mix.Tasks.Clean do
 
   # Loadpaths without checks because compilers may be defined in deps.
   defp loadpaths! do
-    Mix.Task.run "loadpaths", ["--no-elixir-version-check", "--no-deps-check", "--no-archives-check"]
+    Mix.Task.run "loadpaths", ["--no-elixir-version-check", "--no-deps-check"]
     Mix.Task.reenable "loadpaths"
   end
 end

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

@@ -97,7 +97,7 @@ defmodule Mix.Tasks.Compile do
 
   # Loadpaths without checks because compilers may be defined in deps.
   defp loadpaths! do
-    Mix.Task.run "loadpaths", ["--no-elixir-version-check", "--no-deps-check", "--no-archives-check"]
+    Mix.Task.run "loadpaths", ["--no-elixir-version-check", "--no-deps-check"]
     Mix.Task.reenable "loadpaths"
   end
 

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

@@ -14,10 +14,15 @@ defmodule Mix.Tasks.Deps.Check do
   ## Command line options
 
     * `--no-compile` - do not compile dependencies
+    * `--no-archives-check` - do not check archives
 
   """
   @spec run(OptionParser.argv) :: :ok
   def run(args) do
+    unless "--no-archives-check" in args do
+      Mix.Task.run "archive.check", args
+    end
+
     all = Enum.map(loaded(env: Mix.env), &check_lock/1)
 
     _ = prune_deps(all)

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

@@ -37,6 +37,10 @@ defmodule Mix.Tasks.Deps.Compile do
 
   @spec run(OptionParser.argv) :: :ok
   def run(args) do
+    unless "--no-archives-check" in args do
+      Mix.Task.run "archive.check", args
+    end
+
     Mix.Project.get!
 
     case OptionParser.parse(args, switches: @switches) do

lib/mix/lib/mix/tasks/deps.get.ex

@@ -9,10 +9,15 @@ defmodule Mix.Tasks.Deps.Get do
 
   ## Command line options
 
-    * `--only`  - only fetch dependencies for given environment
+    * `--only` - only fetch dependencies for given environment
+    * `--no-archives-check` - do not check archives before fetching deps
   """
   @spec run(OptionParser.argv) :: :ok
   def run(args) do
+    unless "--no-archives-check" in args do
+      Mix.Task.run "archive.check", args
+    end
+
     Mix.Project.get!
     {opts, _, _} = OptionParser.parse(args, switches: [only: :string])
 

lib/mix/lib/mix/tasks/deps.update.ex

@@ -13,11 +13,15 @@ defmodule Mix.Tasks.Deps.Update do
 
   ## Command line options
 
-    * `--all`  - update all dependencies
+    * `--all` - update all dependencies
     * `--only` - only fetch dependencies for given environment
+    * `--no-archives-check` - do not check archives before fetching deps
   """
   @spec run(OptionParser.argv) :: [atom]
   def run(args) do
+    unless "--no-archives-check" in args do
+      Mix.Task.run "archive.check", args
+    end
     Mix.Project.get!
     {opts, rest, _} = OptionParser.parse(args, switches: [all: :boolean, only: :string])
 

lib/mix/lib/mix/tasks/help.ex

@@ -109,7 +109,7 @@ defmodule Mix.Tasks.Help do
 
   # Loadpaths without checks because tasks may be defined in deps.
   defp loadpaths! do
-    Mix.Task.run "loadpaths", ["--no-elixir-version-check", "--no-deps-check", "--no-archives-check"]
+    Mix.Task.run "loadpaths", ["--no-elixir-version-check", "--no-deps-check"]
     Mix.Task.reenable "loadpaths"
   end
 

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

@@ -7,7 +7,7 @@ defmodule Mix.Tasks.Loadpaths do
   ## Command line options
 
     * `--no-archives-check` - do not check archive
-    * `--no-deps-check` - do not check dependencies
+    * `--no-deps-check` - do not check dependencies (also implies --no-archives-check)
     * `--no-elixir-version-check` - do not check Elixir version
 
   """
@@ -20,10 +20,6 @@ defmodule Mix.Tasks.Loadpaths do
       check_elixir_version(config, args)
     end
 
-    unless "--no-archives-check" in args do
-      Mix.Task.run "archive.check", args
-    end
-
     # --no-deps is used only internally. It has not purpose
     # from Mix.CLI because the CLI itself already loads deps.
     unless "--no-deps" in args do