diff options
| author | Tim Watson <watson.timothy@gmail.com> | 2011-09-18 08:27:15 +0100 |
|---|---|---|
| committer | Tuncer Ayaz <tuncer.ayaz@gmail.com> | 2011-10-20 16:31:08 +0200 |
| commit | 86325b616d123bab9517fa957e08f0acb1fffd93 (patch) | |
| tree | 68326df53399633cd384e9577361fc2afe35d7cd | |
| parent | 0ce5cbab0648a6bac526a330bee19f7782f76c7d (diff) | |
Allow plugins to participate in pre and post processing
This patch modifies rebar_core to allow plugins to participate in the
pre and post processing steps, giving plugin authors more flexibility
and control.
| -rw-r--r-- | src/rebar_core.erl | 22 | ||||
| -rw-r--r-- | src/rebar_deps.erl | 1 |
2 files changed, 15 insertions, 8 deletions
diff --git a/src/rebar_core.erl b/src/rebar_core.erl index 832bb32..e457f1f 100644 --- a/src/rebar_core.erl +++ b/src/rebar_core.erl @@ -128,8 +128,18 @@ process_dir(Dir, ParentConfig, Command, DirSet) -> %% Invoke 'preprocess' on the modules -- this yields a list of other %% directories that should be processed _before_ the current one. Predirs = acc_modules(Modules, preprocess, Config, ModuleSetFile), - ?DEBUG("Predirs: ~p\n", [Predirs]), - DirSet2 = process_each(Predirs, Command, Config, + + %% Get the list of plug-in modules from rebar.config. These + %% modules may participate in preprocess and postprocess. + {ok, PluginModules} = plugin_modules(Config), + + PluginPredirs = acc_modules(PluginModules, preprocess, + Config, ModuleSetFile), + + AllPredirs = Predirs ++ PluginPredirs, + + ?DEBUG("Predirs: ~p\n", [AllPredirs]), + DirSet2 = process_each(AllPredirs, Command, Config, ModuleSetFile, DirSet), %% Make sure the CWD is reset properly; processing the dirs may have @@ -144,11 +154,6 @@ process_dir(Dir, ParentConfig, Command, DirSet) -> ?INFO("Skipping ~s in ~s\n", [Command, Dir]); false -> - %% Get the list of plug-in modules from rebar.config. These - %% modules are processed LAST and do not participate - %% in preprocess. - {ok, PluginModules} = plugin_modules(Config), - %% Execute any before_command plugins on this directory execute_pre(Command, PluginModules, Config, ModuleSetFile), @@ -167,7 +172,8 @@ process_dir(Dir, ParentConfig, Command, DirSet) -> %% Invoke 'postprocess' on the modules. This yields a list of other %% directories that should be processed _after_ the current one. - Postdirs = acc_modules(Modules, postprocess, Config, ModuleSetFile), + Postdirs = acc_modules(Modules ++ PluginModules, postprocess, + Config, ModuleSetFile), ?DEBUG("Postdirs: ~p\n", [Postdirs]), DirSet4 = process_each(Postdirs, Command, Config, ModuleSetFile, DirSet3), diff --git a/src/rebar_deps.erl b/src/rebar_deps.erl index 21313e3..2f00760 100644 --- a/src/rebar_deps.erl +++ b/src/rebar_deps.erl @@ -201,6 +201,7 @@ update_deps_code_path([Dep | Rest]) -> {true, _} -> Dir = filename:join(Dep#dep.dir, "ebin"), ok = filelib:ensure_dir(filename:join(Dir, "dummy")), + ?DEBUG("Adding ~s to code path", [Dir]), true = code:add_patha(Dir); {false, _} -> true |