Chromium Code Reviews| Index: scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py |
| diff --git a/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py b/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py |
| index 9e5535c7cb85cd205ef95d5d23e5f18fc29a84f3..2079d8d10d18dc6649d9dc77a3e7a0ebdc552a73 100644 |
| --- a/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py |
| +++ b/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py |
| @@ -53,7 +53,7 @@ def compare_build_artifacts(first_dir, second_dir): |
| if diff: |
| print >> sys.stderr, 'Different list of files in both directories' |
| print >> sys.stderr, '\n'.join(' ' + i for i in sorted(diff)) |
| - res = 1 |
| + res += len(diff) |
| for f in sorted(first_list & second_list): |
| first_file = os.path.join(first_dir, f) |
| @@ -62,9 +62,12 @@ def compare_build_artifacts(first_dir, second_dir): |
| print('%s: equal' % f) |
| else: |
| print('%s: DIFFERENT' % f) |
| - res = 1 |
| + res += 1 |
| - return res |
| + print '%d files are equal, %d are different.' % ( |
| + len(first_list & second_list), res) |
|
M-A Ruel
2014/10/28 20:31:56
calculation is wrong
Sébastien Marchand
2014/10/28 20:36:57
Oops, forgot to save my changes before uploading..
|
| + |
| + return 0 if res == 0 else 1 |
| def main(): |