diff options
| author | Fred Hebert <mononcqc@ferd.ca> | 2014-12-07 21:12:07 +0000 |
|---|---|---|
| committer | Fred Hebert <mononcqc@ferd.ca> | 2014-12-07 21:16:48 +0000 |
| commit | 4116d41dca67b26bef26c925722ab0cdaa850fa8 (patch) | |
| tree | 7673d18643fc6d63f5b3d4279936eb9dee4dc93d /src | |
| parent | ac077e87eed86aedec177548c9a8d382271eb127 (diff) | |
Print warning when deps are being skipped.
The case in mind here is due to conflicts, and tests have been added for
this.
Diffstat (limited to 'src')
| -rw-r--r-- | src/rebar_prv_install_deps.erl | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/rebar_prv_install_deps.erl b/src/rebar_prv_install_deps.erl index 00ef9c5..4f8ea98 100644 --- a/src/rebar_prv_install_deps.erl +++ b/src/rebar_prv_install_deps.erl @@ -206,6 +206,7 @@ update_src_deps(Level, SrcDeps, PkgDeps, SrcApps, State, Update, Seen) -> %% If not seen, add to list of locks to write out case sets:is_element(rebar_app_info:name(AppInfo), SeenAcc) of true -> + warn_skip_deps(AppInfo), {SrcDepsAcc, PkgDepsAcc, SrcAppsAcc, StateAcc, SeenAcc}; false -> {SeenAcc1, StateAcc1} = maybe_lock(AppInfo, SeenAcc, StateAcc), @@ -419,3 +420,9 @@ info(Description) -> {app_name, ".*", {fossil, "https://www.example.org/url", "Vsn"}}, {app_name, ".*", {p4, "//depot/subdir/app_dir"}}]} ]). + +warn_skip_deps(AppInfo) -> + ?WARN("Skipping ~s (from ~p) as an app of the same name " + "has already been fetched~n", + [rebar_app_info:name(AppInfo), + rebar_app_info:source(AppInfo)]). |