No need to store environment in .compile.lock

Author: José Valim

lib/mix/lib/mix/deps/lock.ex

@@ -26,7 +26,7 @@ defmodule Mix.Deps.Lock do
   """
   def touch(manifest_path) do
     File.mkdir_p!(manifest_path)
-    File.write!(Path.join(manifest_path, @manifest), "#{System.version}\n#{Mix.env}")
+    File.write!(Path.join(manifest_path, @manifest), System.version)
   end
 
   @doc """
@@ -40,30 +40,11 @@ defmodule Mix.Deps.Lock do
   Returns the elixir version in the lock manifest in the given path.
   """
   def elixir_vsn(manifest_path) do
-    read_manifest(manifest_path) |> elem(0)
-  end
-
-  @doc """
-  Returns the mix env in the lock manifest unless it is an umbrella app.
-  """
-  def mix_env() do
-    unless Mix.Project.umbrella?, do: mix_env(Mix.Project.manifest_path)
-  end
-
-  @doc """
-  Returns the mix env version in the lock manifest in the given path.
-  """
-  def mix_env(manifest_path) do
-    read_manifest(manifest_path) |> elem(1)
-  end
-
-  defp read_manifest(manifest_path) do
     case File.read(manifest(manifest_path)) do
       { :ok, contents } ->
-        destructure [vsn, env], String.split(contents, "\n")
-        { vsn, env }
+        contents
       { :error, _ } ->
-        { nil, nil }
+        nil
     end
   end
 

lib/mix/test/mix/deps/lock_test.exs

@@ -26,15 +26,11 @@ defmodule Mix.Deps.LockTest do
     end
   end
 
-  test "stores version and env in manifest" do
+  test "stores version in manifest" do
     in_fixture "no_mixfile", fn ->
       assert nil? Mix.Deps.Lock.elixir_vsn
-      assert nil? Mix.Deps.Lock.mix_env
-
       Mix.Deps.Lock.touch
-
       assert Mix.Deps.Lock.elixir_vsn == System.version
-      assert Mix.Deps.Lock.mix_env == "dev"
     end
   end
 end