Allow top-level apps to take precedence over deps

The use case has been described in issue #1478 where a local application
can exist while being declared as a dependency as well. This allows, for
example, to work on a release where all applications may require to be
published independently, or to provide some form of 'vendoring' with a
local app.

The fix is done by decoupling the dependency source resolution form the
dependency parsing. The reason for this being that the discovery phase
needs to parse apps for their top-level deps, and dep installation needs
to resolve the packages with accuracy. In the current implementation,
both code paths call to the same function.

This patch splits up the precise discovery and makes it happen *only*
when installing dependencies, and only if a top-level app does not
already define the application needing resolving.

One weakness of this fix is that it necessarily breaks cycle detection
in dependencies that involve a root application depending on itself
since its own version as a dep will not be expanded. There appears to be
no possible way to prevent this, but should be rare enough to be worth
the tradeoff for the common case.

1 files changed, 29 insertions, 1 deletions

diff --git a/test/rebar_deps_SUITE.erl b/test/rebar_deps_SUITE.erl
index 6b2ecea..ae50ea3 100644
--- a/test/rebar_deps_SUITE.erl
+++ b/test/rebar_deps_SUITE.erl
@@ -7,7 +7,7 @@ all() -> [sub_app_deps, newly_added_dep, newly_added_after_empty_lock,
           http_proxy_settings, https_proxy_settings,
           http_os_proxy_settings, https_os_proxy_settings,
           semver_matching_lt, semver_matching_lte, semver_matching_gt,
-          valid_version, {group, git}, {group, pkg}].
+          valid_version, top_override, {group, git}, {group, pkg}].
 
 groups() ->
     [{all, [], [flat, pick_highest_left, pick_highest_right,
@@ -47,6 +47,8 @@ init_per_testcase(newly_added_dep, Config) ->
     rebar_test_utils:init_rebar_state(Config);
 init_per_testcase(sub_app_deps, Config) ->
     rebar_test_utils:init_rebar_state(Config);
+init_per_testcase(top_override, Config) ->
+    rebar_test_utils:init_rebar_state(Config);
 init_per_testcase(http_proxy_settings, Config) ->
     %% Create private rebar.config
     Priv = ?config(priv_dir, Config),
@@ -255,6 +257,32 @@ circular1(Config) -> run(Config).
 circular2(Config) -> run(Config).
 circular_skip(Config) -> run(Config).
 
+%% Test that a top-level application overtakes dependencies, and
+%% works even if said deps do not exist.
+top_override(Config) ->
+    AppDir = ?config(apps, Config),
+    ct:pal("dir: ~p", [AppDir]),
+    Name1 = rebar_test_utils:create_random_name("sub_app1_"),
+    Name2 = rebar_test_utils:create_random_name("sub_app2_"),
+    SubAppsDir1 = filename:join([AppDir, "apps", Name1]),
+    SubAppsDir2 = filename:join([AppDir, "apps", Name2]),
+    Vsn = rebar_test_utils:create_random_vsn(),
+    rebar_test_utils:create_app(SubAppsDir1, Name1, Vsn, [kernel, stdlib]),
+    rebar_test_utils:create_app(SubAppsDir2, Name2, Vsn, [kernel, stdlib]),
+    rebar_test_utils:create_config(
+      SubAppsDir1,
+      [{deps, [list_to_atom(Name2)]}]
+     ),
+    rebar_test_utils:create_config(
+      SubAppsDir2,
+      [{deps, [{list_to_atom(Name1),
+                {git, "https://example.org", {branch, "master"}}}]}]
+    ),
+    rebar_test_utils:run_and_check(
+      Config, [], ["compile"],
+      {ok, [{app, Name1}, {app,Name2}]}
+    ).
+
 %% Test that the deps of project apps that have their own rebar.config
 %% are included, but that top level rebar.config deps take precedence
 sub_app_deps(Config) ->