From 02fa5f6f42dbeabf324ebc61b049b030a45d30ad Mon Sep 17 00:00:00 2001 From: Dave Smith Date: Thu, 31 Dec 2009 19:29:33 -0700 Subject: [PATCH] Make sure to reset CWD after processing subdirs; also filter out missing dirs when restoring code path --- src/rebar_core.erl | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/rebar_core.erl b/src/rebar_core.erl index b938b4b..c3ad968 100644 --- a/src/rebar_core.erl +++ b/src/rebar_core.erl @@ -162,8 +162,13 @@ process_dir(Dir, ParentConfig, Commands) -> %% are any other dirs that might need processing first. {UpdatedConfig, Dirs} = acc_modules(select_modules(Modules, preprocess, []), preprocess, Config, ModuleSetFile, []), + ?DEBUG("~s subdirs: ~p\n", [Dir, Dirs]), [process_dir(D, UpdatedConfig, Commands) || D <- Dirs], + %% Make sure the CWD is reset properly; processing subdirs may have caused it + %% to change + ok = file:set_cwd(Dir), + %% Finally, process the current working directory apply_commands(Commands, Modules, UpdatedConfig, ModuleSetFile), @@ -239,7 +244,9 @@ update_code_path(Config) -> restore_code_path(no_change) -> ok; restore_code_path({old, Path}) -> - true = code:set_path(Path), + %% Verify that all of the paths still exist -- some dynamically add paths + %% can get blown away during clean. + true = code:set_path(lists:filter(fun filelib:is_file/1, Path)), ok.