Merge pull request #1264 from edgurgel/mix-rebar-scripts

Fix rebar config scripts evaluation

Author: ericmj

lib/mix/lib/mix/rebar.ex

@@ -128,14 +128,17 @@ defmodule Mix.Rebar do
     end
   end
 
-  defp eval_script(path, config) do
-    script = Path.basename(path) |> binary_to_list
-    case :file.script(path, eval_binds(CONFIG: config, SCRIPT: script)) do
+  defp eval_script(script_path, config) do
+    script = Path.basename(script_path) |> binary_to_list
+    result = File.cd!(Path.dirname(script_path), fn ->
+      :file.script(script, eval_binds(CONFIG: config, SCRIPT: script))
+    end)
+    case result do
       { :ok, config } ->
         config
       { :error, error } ->
         reason = :file.format_error(error)
-        raise Mix.Error, message: "Error evaluating rebar config script #{path}: #{reason}"
+        raise Mix.Error, message: "Error evaluating rebar config script #{script_path}: #{reason}"
     end
   end
 

lib/mix/test/fixtures/rebar_dep_script/rebar.config

@@ -0,0 +1 @@
+[{'dir', file:get_cwd()}].

lib/mix/test/fixtures/rebar_dep_script/rebar.config.script

@@ -21,6 +21,12 @@ defmodule Mix.RebarTest do
     assert config[:SCRIPT] == 'rebar.config.script'
   end
 
+  test "execute rebar.config.script on dependecy directory" do
+    path = MixTest.Case.fixture_path("rebar_dep_script")
+    config = Mix.Rebar.load_config(path)
+    assert config[:dir] == {:ok, binary_to_list(path)}
+  end
+
   test "rebar sub_dirs" do
     path = MixTest.Case.fixture_path("rebar_dep")
     assert Mix.Rebar.recur(path, fn _ -> File.cwd end)