From 8c03d5959be4f9e14ed72bbaa3d17d0fc67412fb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Oscar=20Hellstr=C3=B6m?= Date: Tue, 24 Aug 2010 21:54:54 +0200 Subject: [PATCH] Make sure *both* .eunit and ebin directory exists before setting up the code path. Normally the ebin directory doesn't contain any source files. Therefore it won't be kept in the repository by, at least, mercurial and also maybe git unless you put some .keep file in it or do some other hack. The ebin directory is created by rebar compile, but if rebar eunit is called before rebar compile, you end up with a {'EXIT', {{badmatch,{error,bad_directory}},...}. Another approach would be not to match cod:add_pathz(ebin_dir()) with ok, but I think this is an ok solution as well. --- src/rebar_eunit.erl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/rebar_eunit.erl b/src/rebar_eunit.erl index 9f2baf1..a8ce954 100644 --- a/src/rebar_eunit.erl +++ b/src/rebar_eunit.erl @@ -74,8 +74,9 @@ eunit(Config, AppFile) -> end end, - %% Make sure ?EUNIT_DIR/ directory exists (tack on dummy module) + %% Make sure ?EUNIT_DIR/ and ebin/ directory exists (tack on dummy module) ok = filelib:ensure_dir(?EUNIT_DIR ++ "/foo"), + ok = filelib:ensure_dir(ebin_dir() ++ "/foo"), %% Setup code path prior to compilation so that parse_transforms and the like %% work properly. Also, be sure to add ebin_dir() to the END of the code path