Also shows detailed diffs on expected diffs.

tools/determinism/compare_build_artifacts.py

Index: tools/determinism/compare_build_artifacts.py
diff --git a/tools/determinism/compare_build_artifacts.py b/tools/determinism/compare_build_artifacts.py
index 769e73abb1c5f63c07fead8ff85cce3172a3b05a..ad95445c430870c9d5d9c660d40ada6807666a5d 100755
--- a/tools/determinism/compare_build_artifacts.py
+++ b/tools/determinism/compare_build_artifacts.py
@@ -625,8 +625,9 @@ def compare_build_artifacts(first_dir, second_dir, target_platform,
     for u in unexpected_equals:
       print('  %s' % u)
 
-  if unexpected_diffs:
-    diffs_to_investigate = set(unexpected_diffs).difference(
+  if expected_diffs or unexpected_diffs:
+    all_diffs = expected_diffs + unexpected_diffs
+    diffs_to_investigate = set(all_diffs).difference(
         set(missing_files))
     compare_deps(first_dir, second_dir, diffs_to_investigate)

[Dirk Pranke, 2016/08/29 15:51:34]

Nit: You don't actually need the `if` check on line 628 at all; lines 629-632
will do the
right thing even if run unconditionally, so I would eliminate the branch.

[Yoshisato Yanagisawa, 2016/08/30 01:29:49]

done.
also removed unnecessary set(), and make diffs sorted.