github/rebar
1
0
Fork 0
mirror of https://github.com/correl/rebar.git synced 2024-11-23 19:19:54 +00:00
Code Issues Projects Releases Packages Wiki Activity

Update comment for filelib:ensure_dir/1 wrapper.

Browse source 
The wrapper can now be removed as I've fixed the problem in R13B04.
This commit is contained in:
Tuncer Ayaz 2010-02-17 23:38:06 +01:00
parent b0dfd233ae
commit 00c855d030
1 changed files with 1 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/rebar_utils.erl
View file

@ -96,14 +96,7 @@ now_str() ->
lists:flatten(io_lib:format("~4b/~2..0b/~2..0b ~2..0b:~2..0b:~2..0b", lists:flatten(io_lib:format("~4b/~2..0b/~2..0b ~2..0b:~2..0b:~2..0b",
[Year, Month, Day, Hour, Minute, Second])). [Year, Month, Day, Hour, Minute, Second])).
%% TODO: Review why filelib:ensure_dir/1 sometimes returns {error, eexist}. %% TODO: filelib:ensure_dir/1 corrected in R13B04. Can be removed.
%% 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
ensure_dir(Path) -> ensure_dir(Path) ->
case filelib:ensure_dir(Path) of case filelib:ensure_dir(Path) of
ok -> ok ->