Do not rewrite .app file if it didn't change

basho · Aug 7, 2012 · f2f294d · f2f294d
1 parent ca8d7cc
commit f2f294d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/rebar_file_utils.erl b/src/rebar_file_utils.erl
@@ -29,7 +29,8 @@
 -export([rm_rf/1,
          cp_r/2,
          mv/2,
-         delete_each/1]).
+         delete_each/1,
+         write_file_if_contents_differ/2]).
 
 -include("rebar.hrl").
 
@@ -111,6 +112,17 @@ delete_each([File | Rest]) ->
             ?FAIL
     end.
 
+write_file_if_contents_differ(Filename, Bytes) ->
+    ToWrite = iolist_to_binary(Bytes),
+    case file:read_file(Filename) of
+        {ok, ToWrite} ->
+            ok;
+        {ok,  _} ->
+            file:write_file(Filename, ToWrite);
+        {error,  _} ->
+            file:write_file(Filename, ToWrite)
+    end.
+
 %% ===================================================================
 %% Internal functions
 %% ===================================================================

diff --git a/src/rebar_otp_app.erl b/src/rebar_otp_app.erl
@@ -106,7 +106,7 @@ preprocess(Config, AppSrcFile) ->
 
             %% Setup file .app filename and write new contents
             AppFile = rebar_app_utils:app_src_to_app(AppSrcFile),
-            ok = file:write_file(AppFile, Spec),
+            ok = rebar_file_utils:write_file_if_contents_differ(AppFile, Spec),
 
             %% Make certain that the ebin/ directory is available
             %% on the code path