소스 검색

Fix provider hook handling for escriptize

Previously all escriptize hooks from all apps in the project were
executed, which could lead to issue in case of dependencies on artifacts
which are created by these hooks.

Now the behaviour is similar to the handling of other non-global hooks.
pull/2027/head
Tino Breddin 6 년 전
부모
커밋
9c28e9122d
1개의 변경된 파일8개의 추가작업 그리고 7개의 파일을 삭제
  1. +8
    -7
      src/rebar_prv_escriptize.erl

+ 8
- 7
src/rebar_prv_escriptize.erl 파일 보기

@ -63,13 +63,11 @@ desc() ->
do(State) ->
Providers = rebar_state:providers(State),
Cwd = rebar_state:dir(State),
rebar_hooks:run_project_and_app_hooks(Cwd, pre, ?PROVIDER, Providers, State),
?INFO("Building escript...", []),
Res = case rebar_state:get(State, escript_main_app, undefined) of
AppInfo0 = case rebar_state:get(State, escript_main_app, undefined) of
undefined ->
case rebar_state:project_apps(State) of
[App] ->
escriptize(State, App);
[AppInfo] ->
AppInfo;
_ ->
?PRV_ERROR(no_main_app)
end;
@ -77,12 +75,15 @@ do(State) ->
AllApps = rebar_state:all_deps(State)++rebar_state:project_apps(State),
case rebar_app_utils:find(rebar_utils:to_binary(Name), AllApps) of
{ok, AppInfo} ->
escriptize(State, AppInfo);
AppInfo;
_ ->
?PRV_ERROR({bad_name, Name})
end
end,
rebar_hooks:run_project_and_app_hooks(Cwd, post, ?PROVIDER, Providers, State),
AppInfo1 = rebar_hooks:run_all_hooks(Cwd, pre, ?PROVIDER, Providers, AppInfo0, State),
?INFO("Building escript...", []),
Res = escriptize(State, AppInfo1),
rebar_hooks:run_all_hooks(Cwd, post, ?PROVIDER, Providers, AppInfo1, State),
Res.
escriptize(State0, App) ->

불러오는 중...
취소
저장