From 49c25642b3da6796c05385b76160178e3c9a2c83 Mon Sep 17 00:00:00 2001 From: Slava Yurin Date: Thu, 29 May 2014 18:43:23 +0700 Subject: [PATCH] Fix #249 (erlc regression) The combination of changes to rebar_erlc_compiler, and the fact that erl_first_files is inherited, caused a regression. To fix that, ensure every project uses its own .rebar/erlcinfo. While at it, fix the issue that erl_first_files entries were not included when initializing the dep digraph. Reported-by: Louis-Philippe Gauthier Reported-by: Roland Karlsson Thanks: Tuncer Ayaz --- inttest/erlc/rebar-no_debug_info.config | 2 +- inttest/erlc/rebar.config | 3 ++- rebar.config.sample | 2 +- src/rebar_erlc_compiler.erl | 29 +++++++++++++++++++------ 4 files changed, 26 insertions(+), 10 deletions(-) diff --git a/inttest/erlc/rebar-no_debug_info.config b/inttest/erlc/rebar-no_debug_info.config index 1a7113a..07b6fed 100644 --- a/inttest/erlc/rebar-no_debug_info.config +++ b/inttest/erlc/rebar-no_debug_info.config @@ -1,6 +1,6 @@ %% -*- mode: erlang;erlang-indent-level: 4;indent-tabs-mode: nil -*- %% ex: ts=4 sw=4 ft=erlang et -{erl_first_files, ["first_xrl.erl", "first_yrl.erl"]}. +{erl_first_files, ["src/first_xrl.erl", "src/first_yrl.erl"]}. {erl_opts, [ diff --git a/inttest/erlc/rebar.config b/inttest/erlc/rebar.config index 7ea78e1..71d6660 100644 --- a/inttest/erlc/rebar.config +++ b/inttest/erlc/rebar.config @@ -1,6 +1,7 @@ %% -*- mode: erlang;erlang-indent-level: 4;indent-tabs-mode: nil -*- %% ex: ts=4 sw=4 ft=erlang et -{erl_first_files, ["first_xrl.erl", "first_yrl.erl", "src/first_erl.erl"]}. +{erl_first_files, + ["src/first_xrl.erl", "src/first_yrl.erl", "src/first_erl.erl"]}. {deps, [foobar]}. diff --git a/rebar.config.sample b/rebar.config.sample index 515ed00..03d2e98 100644 --- a/rebar.config.sample +++ b/rebar.config.sample @@ -21,7 +21,7 @@ %% Erlang files to compile before the rest. Rebar automatically compiles %% parse_transforms and custom behaviours before anything other than the files %% in this list. -{erl_first_files, ["mymib1", "mymib2"]}. +{erl_first_files, ["src/mymib1.erl", "src/mymib2.erl"]}. %% Erlang compiler options {erl_opts, [no_debug_info, diff --git a/src/rebar_erlc_compiler.erl b/src/rebar_erlc_compiler.erl index b797137..7df488a 100644 --- a/src/rebar_erlc_compiler.erl +++ b/src/rebar_erlc_compiler.erl @@ -208,7 +208,7 @@ info_help(Description) -> "(linux|solaris|freebsd|darwin)", 'HAVE_SENDFILE'}, {platform_define, "(linux|freebsd)", 'BACKLOG', 128}, {platform_define, "R13", 'old_inets'}]}, - {erl_first_files, ["mymib1", "mymib2"]}, + {erl_first_files, ["src/mymib1.erl", "src/mymib2.erl"]}, {mib_opts, []}, {mib_first_files, []}, {xrl_opts, []}, @@ -222,6 +222,13 @@ test_compile_config_and_opts(Config, ErlOpts, Cmd) -> {Config2, PropErOpts} = proper_opts(Config1), {Config3, EqcOpts} = eqc_opts(Config2), + %% NOTE: For consistency, all *_first_files lists should be + %% retrieved via rebar_config:get_local. Right now + %% erl_first_files, eunit_first_files, and qc_first_files use + %% rebar_config:get_list and are inherited, but xrl_first_files + %% and yrl_first_files use rebar_config:get_local. Inheritance of + %% *_first_files is questionable as the file would need to exist + %% in all project directories for it to work. OptsAtom = list_to_atom(Cmd ++ "_compile_opts"), TestOpts = rebar_config:get_list(Config3, OptsAtom, []), Opts0 = [{d, 'TEST'}] ++ @@ -277,20 +284,28 @@ doterl_compile(Config, OutDir) -> doterl_compile(Config, OutDir, [], ErlOpts). doterl_compile(Config, OutDir, MoreSources, ErlOpts) -> - ErlFirstFiles = rebar_config:get_list(Config, erl_first_files, []), + ErlFirstFilesConf = rebar_config:get_list(Config, erl_first_files, []), ?DEBUG("erl_opts ~p~n", [ErlOpts]), %% Support the src_dirs option allowing multiple directories to %% contain erlang source. This might be used, for example, should %% eunit tests be separated from the core application source. SrcDirs = rebar_utils:src_dirs(proplists:append_values(src_dirs, ErlOpts)), - RestErls = [Source || Source <- gather_src(SrcDirs, []) ++ MoreSources, - not lists:member(Source, ErlFirstFiles)], + AllErlFiles = gather_src(SrcDirs, []) ++ MoreSources, + %% NOTE: If and when erl_first_files is not inherited anymore + %% (rebar_config:get_local instead of rebar_config:get_list), consider + %% logging a warning message for any file listed in erl_first_files which + %% wasn't found via gather_src. + {ErlFirstFiles, RestErls} = + lists:partition( + fun(Source) -> + lists:member(Source, ErlFirstFilesConf) + end, AllErlFiles), %% Make sure that ebin/ exists and is on the path ok = filelib:ensure_dir(filename:join("ebin", "dummy.beam")), CurrPath = code:get_path(), true = code:add_path(filename:absname("ebin")), OutDir1 = proplists:get_value(outdir, ErlOpts, OutDir), - G = init_erlcinfo(Config, RestErls), + G = init_erlcinfo(Config, AllErlFiles), %% Split RestErls so that files which are depended on are treated %% like erl_first_files. {OtherFirstErls, OtherErls} = @@ -387,8 +402,8 @@ check_erlcinfo(Config, _) -> ?ABORT("~s file is invalid. Please delete before next run.~n", [erlcinfo_file(Config)]). -erlcinfo_file(Config) -> - filename:join([rebar_utils:base_dir(Config), ".rebar", ?ERLCINFO_FILE]). +erlcinfo_file(_Config) -> + filename:join([rebar_utils:get_cwd(), ".rebar", ?ERLCINFO_FILE]). init_erlcinfo(Config, Erls) -> G = restore_erlcinfo(Config),