Skip to content

Commit

Permalink
External deps_dir should have higher priority than the config one
Browse files Browse the repository at this point in the history
The external deps_dir should have higher priority because
it is used by scripts and other build tools to set up the
location of the dependencies. This commit ensures that,
even if a project has set deps_dir in its config file has
lower preference than the command line one.
  • Loading branch information
José Valim committed Jun 17, 2013
1 parent 16eed82 commit e467642
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/rebar_deps.erl
Expand Up @@ -241,10 +241,11 @@ info_help(Description) ->

%% Added because of trans deps,
%% need all deps in same dir and should be the one set by the root rebar.config
%% In case one is given globally, it has higher priority
%% Sets a default if root config has no deps_dir set
set_shared_deps_dir(Config, []) ->
GlobalDepsDir = rebar_config:get_global(Config, deps_dir, "deps"),
DepsDir = rebar_config:get_local(Config, deps_dir, GlobalDepsDir),
LocalDepsDir = rebar_config:get_local(Config, deps_dir, "deps"),
DepsDir = rebar_config:get_global(Config, deps_dir, LocalDepsDir),
rebar_config:set_xconf(Config, deps_dir, DepsDir);
set_shared_deps_dir(Config, _DepsDir) ->
Config.
Expand Down

0 comments on commit e467642

Please sign in to comment.