From 00c855d030123dcdced313aa388fecf5740a8a35 Mon Sep 17 00:00:00 2001 From: Tuncer Ayaz Date: Wed, 17 Feb 2010 23:38:06 +0100 Subject: [PATCH] Update comment for filelib:ensure_dir/1 wrapper. The wrapper can now be removed as I've fixed the problem in R13B04. --- src/rebar_utils.erl | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/rebar_utils.erl b/src/rebar_utils.erl index 32a2261..bfe7bce 100644 --- a/src/rebar_utils.erl +++ b/src/rebar_utils.erl @@ -96,14 +96,7 @@ now_str() -> lists:flatten(io_lib:format("~4b/~2..0b/~2..0b ~2..0b:~2..0b:~2..0b", [Year, Month, Day, Hour, Minute, Second])). -%% TODO: Review why filelib:ensure_dir/1 sometimes returns {error, eexist}. -%% There appears to be a race condition when calling ensure_dir from -%% multiple processes simultaneously. -%% This does not happen with -j1 but with anything higher than that. -%% So -j2 or default jobs setting will reveal the issue. -%% To reproduce make sure that the priv/mibs directory does not exist -%% $ rm -r priv -%% $ ./rebar -v compile +%% TODO: filelib:ensure_dir/1 corrected in R13B04. Can be removed. ensure_dir(Path) -> case filelib:ensure_dir(Path) of ok ->