Ver a proveniência

include providers listed in plugins env

pull/76/head
Tristan Sloughter há 10 anos
ascendente
cometimento
b4c579b80d
2 ficheiros alterados com 19 adições e 14 eliminações
  1. +0
    -1
      src/rebar.app.src
  2. +19
    -13
      src/rebar_plugins.erl

+ 0
- 1
src/rebar.app.src Ver ficheiro

@ -21,7 +21,6 @@
%% Default log level
{log_level, warn},
%% any_dir processing modules
{providers, [rebar_prv_as,
rebar_prv_clean,
rebar_prv_deps,

+ 19
- 13
src/rebar_plugins.erl Ver ficheiro

@ -18,9 +18,9 @@ install(State) ->
DepsDir = rebar_dir:deps_dir(State1),
expand_plugins(DepsDir),
Plugins = rebar_state:get(State1, plugins, []),
PluginProviders = rebar_utils:filtermap(fun(Plugin) ->
handle_plugin(Plugin, State1)
end, Plugins),
PluginProviders = lists:flatten(rebar_utils:filtermap(fun(Plugin) ->
handle_plugin(Plugin, State1)
end, Plugins)),
State2 = rebar_state:set(State1, deps_dir, OldDepsDir),
{ok, PluginProviders, State2}.
@ -55,16 +55,22 @@ plugin_providers(Plugin) when is_atom(Plugin) ->
validate_plugin(Plugin).
validate_plugin(Plugin) ->
Exports = sets:from_list(Plugin:module_info(exports)),
Required = sets:from_list([{init,1},
{do,1},
{format_error,1}]),
case sets:is_subset(Required, Exports) of
false ->
?WARN("Plugin ~p is not a provider. It will not be used.~n", [Plugin]),
false;
true ->
{true, Plugin}
application:load(Plugin),
case application:get_env(Plugin, providers) of
{ok, Providers} ->
{true, Providers};
undefined ->
Exports = sets:from_list(Plugin:module_info(exports)),
Required = sets:from_list([{init,1},
{do,1},
{format_error,1}]),
case sets:is_subset(Required, Exports) of
false ->
?WARN("Plugin ~p is not a provider. It will not be used.~n", [Plugin]),
false;
true ->
{true, Plugin}
end
end.
expand_plugins(Dir) ->

Carregando…
Cancelar
Guardar