[erlang-patches] erl -make should set exit code on failure

Magnus Henoch magnus@REDACTED
Wed Mar 18 18:04:03 CET 2009


This is just something I need to get off my chest...

It always bothered me that "erl -make" would return exit code 0
regardless of whether the compilation succeeded or failed.  So here is a
patch (against R13A) that sets exit code 1 on failure.  I hope it will
be useful to someone.

--- otp_src_R13A/erts/etc/common/erlexec.c	2009/03/18 16:37:47	1.1
+++ otp_src_R13A/erts/etc/common/erlexec.c	2009/03/18 16:40:23	1.2
@@ -678,7 +678,7 @@
 		     * on itself here.  We'll avoid doing that.
 		     */
 		    if (strcmp(argv[i], "-make") == 0) {
-			add_args("-noshell", "-noinput", "-s", "make", "all", NULL);
+			add_args("-noshell", "-noinput", "-s", "make", "all_or_nothing", NULL);
 			add_Eargs("-B");
 			haltAfterwards = 1;
 			i = argc; /* Skip rest of command line */
--- otp_src_R13A/lib/tools/src/make.erl	2009/03/18 16:38:24	1.1
+++ otp_src_R13A/lib/tools/src/make.erl	2009/03/18 16:39:46	1.2
@@ -24,12 +24,20 @@
 %% If Emakefile is missing the current directory is used.
 -module(make).
 
--export([all/0,all/1,files/1,files/2]).
+-export([all_or_nothing/0,all/0,all/1,files/1,files/2]).
 
 -include_lib("kernel/include/file.hrl").
 
 -define(MakeOpts,[noexec,load,netload,noload]).
 
+all_or_nothing() ->
+    case all() of
+        up_to_date ->
+            up_to_date;
+        error ->
+            halt(1)
+    end.
+
 all() ->
     all([]).
 


-- 
Magnus Henoch, magnus@REDACTED
Erlang Training and Consulting
http://www.erlang-consulting.com/




More information about the erlang-patches mailing list