From 3102f456473f42b170b0c1a47786831b712b1b08 Mon Sep 17 00:00:00 2001 From: Eric Date: Tue, 18 Dec 2012 09:59:34 -0500 Subject: fix a bug in the ability to rebuild releases --- src/rcl_prv_assembler.erl | 24 +++++++++++++++++++++--- 1 file changed, 21 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/rcl_prv_assembler.erl b/src/rcl_prv_assembler.erl index 75a54ec..1ba31f8 100644 --- a/src/rcl_prv_assembler.erl +++ b/src/rcl_prv_assembler.erl @@ -79,7 +79,11 @@ format_error({unable_to_create_output_dir, OutputDir}) -> [OutputDir]); format_error({release_script_generation_error, Module, Errors}) -> ["Errors generating release \n", - rcl_util:indent(1), Module:format_error(Errors)]. + rcl_util:indent(1), Module:format_error(Errors)]; +format_error({unable_to_make_symlink, AppDir, TargetDir, Reason}) -> + io_lib:format("Unable to symlink directory ~s to ~s because \n~s~s", + [AppDir, TargetDir, rcl_util:indent(1), + file:format_error(Reason)]). %%%=================================================================== %%% Internal Functions @@ -125,7 +129,7 @@ copy_app(LibDir, App) -> TargetDir = filename:join([LibDir, AppName ++ "-" ++ AppVsn]), case rcl_app_info:link(App) of true -> - file:make_symlink(AppDir, TargetDir); + link_directory(AppDir, TargetDir); false -> ec_plists:map(fun(SubDir) -> copy_dir(AppDir, TargetDir, SubDir) @@ -138,6 +142,20 @@ copy_app(LibDir, App) -> "LICENSE"]) end. +link_directory(AppDir, TargetDir) -> + case ec_file:exists(TargetDir) of + true -> + ec_file:remove(TargetDir); + false -> + ok + end, + case file:make_symlink(AppDir, TargetDir) of + {error, Reason} -> + ?RCL_ERROR({unable_to_make_symlink, AppDir, TargetDir, Reason}); + ok -> + ok + end. + copy_dir(AppDir, TargetDir, SubDir) -> SubSource = filename:join(AppDir, SubDir), SubTarget = filename:join(TargetDir, SubDir), @@ -167,7 +185,7 @@ create_release_info(State, Release, OutputDir) -> rcl_release:vsn(Release)]), ReleaseFile = filename:join([ReleaseDir, RelName ++ ".rel"]), ok = ec_file:mkdir_p(ReleaseDir), - case rcl_release:metadata(Release) of + case rcl_release:metadata(Release) of {ok, Meta} -> ok = ec_file:write_term(ReleaseFile, Meta), write_bin_file(State, Release, OutputDir, ReleaseDir); -- cgit v1.2.3 From 19d05486ac45580e139501fde52f73c9e43eefc0 Mon Sep 17 00:00:00 2001 From: Eric Date: Tue, 18 Dec 2012 11:15:38 -0500 Subject: make sure that copy errors can be correctly detected --- src/rcl_prv_assembler.erl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/rcl_prv_assembler.erl b/src/rcl_prv_assembler.erl index 1ba31f8..99b5eab 100644 --- a/src/rcl_prv_assembler.erl +++ b/src/rcl_prv_assembler.erl @@ -112,9 +112,9 @@ copy_app_directories_to_output(State, Release, OutputDir) -> (_) -> false end, - ec_plists:map(fun(App) -> - copy_app(LibDir, App) - end, Apps)), + lists:flatten(ec_plists:map(fun(App) -> + copy_app(LibDir, App) + end, Apps))), case Result of [E | _] -> E; -- cgit v1.2.3 From 51b627c68792039c7c7162d354ff0250c7a795b8 Mon Sep 17 00:00:00 2001 From: Eric Date: Tue, 18 Dec 2012 12:01:29 -0500 Subject: make sure that the app target dir is created befor a copy --- src/rcl_prv_assembler.erl | 1 + 1 file changed, 1 insertion(+) (limited to 'src') diff --git a/src/rcl_prv_assembler.erl b/src/rcl_prv_assembler.erl index 99b5eab..271b0d5 100644 --- a/src/rcl_prv_assembler.erl +++ b/src/rcl_prv_assembler.erl @@ -131,6 +131,7 @@ copy_app(LibDir, App) -> true -> link_directory(AppDir, TargetDir); false -> + ok = rcl_util:mkdir_p(TargetDir), ec_plists:map(fun(SubDir) -> copy_dir(AppDir, TargetDir, SubDir) end, ["ebin", -- cgit v1.2.3 From 16368e6531d6bdcd6a2f6a92bd18a20c131766f0 Mon Sep 17 00:00:00 2001 From: Eric Date: Tue, 18 Dec 2012 16:44:44 -0500 Subject: ensure that each directory is explicitly created before a copy occures. --- src/rcl_prv_assembler.erl | 1 + 1 file changed, 1 insertion(+) (limited to 'src') diff --git a/src/rcl_prv_assembler.erl b/src/rcl_prv_assembler.erl index 271b0d5..b8766df 100644 --- a/src/rcl_prv_assembler.erl +++ b/src/rcl_prv_assembler.erl @@ -168,6 +168,7 @@ copy_dir(AppDir, TargetDir, SubDir) -> false -> ok end, + ok = filelib:ensure_dir(SubTarget), case ec_file:copy(SubSource, SubTarget, [recursive]) of {error, E} -> ?RCL_ERROR({ec_file_error, AppDir, TargetDir, E}); -- cgit v1.2.3 From 3cbd64cce8ce1df48f1982819297bb3ba63c0191 Mon Sep 17 00:00:00 2001 From: Eric Date: Fri, 21 Dec 2012 10:42:57 -0500 Subject: refactor rcl_prv_assembler for symplicity and reliability --- src/rcl_prv_assembler.erl | 48 +++++++++++++++++++++++++---------------------- 1 file changed, 26 insertions(+), 22 deletions(-) (limited to 'src') diff --git a/src/rcl_prv_assembler.erl b/src/rcl_prv_assembler.erl index b8766df..a721b12 100644 --- a/src/rcl_prv_assembler.erl +++ b/src/rcl_prv_assembler.erl @@ -127,29 +127,28 @@ copy_app(LibDir, App) -> AppVsn = rcl_app_info:vsn_as_string(App), AppDir = rcl_app_info:dir(App), TargetDir = filename:join([LibDir, AppName ++ "-" ++ AppVsn]), + remove_symlink_or_directory(TargetDir), case rcl_app_info:link(App) of true -> link_directory(AppDir, TargetDir); false -> - ok = rcl_util:mkdir_p(TargetDir), - ec_plists:map(fun(SubDir) -> - copy_dir(AppDir, TargetDir, SubDir) - end, ["ebin", - "include", - "priv", - "src", - "c_src", - "README", - "LICENSE"]) + copy_directory(AppDir, TargetDir) end. -link_directory(AppDir, TargetDir) -> - case ec_file:exists(TargetDir) of +remove_symlink_or_directory(TargetDir) -> + case ec_file:is_symlink(TargetDir) of true -> ec_file:remove(TargetDir); false -> - ok - end, + case filelib:is_dir(TargetDir) of + true -> + ok = ec_file:remove(TargetDir, [recursive]); + false -> + ok + end + end. + +link_directory(AppDir, TargetDir) -> case file:make_symlink(AppDir, TargetDir) of {error, Reason} -> ?RCL_ERROR({unable_to_make_symlink, AppDir, TargetDir, Reason}); @@ -157,21 +156,26 @@ link_directory(AppDir, TargetDir) -> ok end. +copy_directory(AppDir, TargetDir) -> + ec_plists:map(fun(SubDir) -> + copy_dir(AppDir, TargetDir, SubDir) + end, ["ebin", + "include", + "priv", + "src", + "c_src", + "README", + "LICENSE"]). + copy_dir(AppDir, TargetDir, SubDir) -> SubSource = filename:join(AppDir, SubDir), SubTarget = filename:join(TargetDir, SubDir), case filelib:is_dir(SubSource) of true -> - case filelib:is_dir(SubTarget) of - true -> - ok = ec_file:remove(SubTarget, [recursive]); - false -> - ok - end, - ok = filelib:ensure_dir(SubTarget), + ok = rcl_util:mkdir_p(SubTarget), case ec_file:copy(SubSource, SubTarget, [recursive]) of {error, E} -> - ?RCL_ERROR({ec_file_error, AppDir, TargetDir, E}); + ?RCL_ERROR({ec_file_error, AppDir, SubTarget, E}); ok -> ok end; -- cgit v1.2.3 From a9b6d65069ceac9b601216c10a0e60c97a80603a Mon Sep 17 00:00:00 2001 From: Eric Date: Fri, 21 Dec 2012 10:43:09 -0500 Subject: fix nasty bug in discovery process --- src/rcl_cmd_args.erl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/rcl_cmd_args.erl b/src/rcl_cmd_args.erl index d72bce3..dc176be 100644 --- a/src/rcl_cmd_args.erl +++ b/src/rcl_cmd_args.erl @@ -145,7 +145,7 @@ convert_goals([RawSpec | Rest], Acc) -> {ok, rcl_state:cmd_args()} | relcool:error(). create_output_dir(Opts, Acc) -> OutputDir = proplists:get_value(output_dir, Opts, "./_rel"), - create_lib_dirs(Opts, [{output_dir, OutputDir} | Acc]). + create_lib_dirs(Opts, [{output_dir, filename:absname(OutputDir)} | Acc]). -spec create_lib_dirs([getopt:option()], rcl_state:cmd_args()) -> {ok, rcl_state:cmd_args()} | relcool:error(). -- cgit v1.2.3