run the http provider before attempting to compile if the package was…

… pulled from hex

lib/mix/tasks/firmware.ex

@@ -35,13 +35,24 @@ defmodule Mix.Tasks.Firmware do
 
     firmware_config = Application.get_env(:nerves, :firmware)
 
-    system_path = System.get_env("NERVES_SYSTEM") || raise """
+    system_path = System.get_env("NERVES_SYSTEM") || Mix.raise """
       Environment variable $NERVES_SYSTEM is not set
     """
 
-    System.get_env("NERVES_TOOLCHAIN") || raise """
+    System.get_env("NERVES_TOOLCHAIN") || Mix.raise """
       Environment variable $NERVES_TOOLCHAIN is not set
     """
+
+    rel_config =
+      File.cwd!
+      |> Path.join("rel/config.exs")
+
+    unless File.exists?(rel_config) do
+      Mix.raise """
+        You are missing a release config file. Run  nerves.release.init task first
+      """
+    end
+
     Mix.Task.run "compile", []
     Mix.Task.run "release", ["--verbosity=#{verbosity}"]
 

lib/nerves/env.ex

@@ -17,7 +17,7 @@ defmodule Nerves.Env do
   """
   @spec start() :: Agent.on_start
   def start do
-    Agent.start_link fn -> load_packages() end, name: __MODULE__
+    Agent.start_link(fn -> load_packages() end, name: __MODULE__)
   end
 
   @doc """

lib/nerves/package.ex

@@ -92,8 +92,19 @@ defmodule Nerves.Package do
   """
   @spec artifact(Nerves.Package.t, Nerves.Package.t) :: :ok
   def artifact(pkg, toolchain) do
-    {mod, opts} = pkg.provider
-    if mod.artifact(pkg, toolchain, opts) == :ok do
+    ret =
+      case pkg.provider do
+        {mod, opts} -> mod.artifact(pkg, toolchain, opts)
+        providers when is_list(providers) ->
+          Enum.reduce(providers, nil, fn ({mod, opts}, ret) ->
+            if ret != :ok do
+              mod.artifact(pkg, toolchain, opts)
+            else
+              ret
+            end
+          end)
+      end
+    if ret == :ok do
       Path.join(Artifact.dir(pkg, toolchain), @checksum)
       |> File.write!(checksum(pkg))
     end
@@ -216,7 +227,7 @@ defmodule Nerves.Package do
     config = Mix.Project.config[:artifacts] || []
 
     case Keyword.get(config, app) do
-      nil -> {provider_mod(type), []}
+      nil -> [{Providers.HTTP, []}, {provider_mod(type), []}]
       opts -> provider_opts(opts)
     end
   end
@@ -254,15 +265,14 @@ defmodule Nerves.Package do
   end
 
   defp dep_type(pkg) do
-    deps_paths = Mix.Project.deps_paths
+    deps_paths = Mix.Project.deps_paths()
     case Map.get(deps_paths, pkg) do
       nil ->
         :project
       path ->
         deps_path =
           File.cwd!
           |> Path.join(Mix.Project.config[:deps_path])
-          |> Path.expand
         if String.starts_with?(path, deps_path) do
           :hex
         else

lib/nerves/package/providers/http.ex

@@ -62,7 +62,10 @@ defmodule Nerves.Package.Providers.HTTP do
 
   defp unpack({:error, _} = error, _, _), do: error
   defp unpack({:ok, tar}, artifact, destination) do
-    shell_info "Unpacking #{artifact}"
+    shell_info """
+      Unpacking #{artifact}
+      To #{destination}
+    """
     tmp_path = Path.join(destination, ".tmp")
     File.mkdir_p!(tmp_path)
     tar_file = Path.join(tmp_path, artifact)