Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(366)

Side by Side Diff: tools/determinism/compare_build_artifacts.py

Issue 2496903002: Also whitelist ppapi_nacl_tests_* on Linux. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2014 The Chromium Authors. All rights reserved. 2 # Copyright 2014 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Compare the artifacts from two builds.""" 6 """Compare the artifacts from two builds."""
7 7
8 import difflib 8 import difflib
9 import json 9 import json
10 import optparse 10 import optparse
(...skipping 16 matching lines...) Expand all
27 # TODO(sebmarchand): Remove this once all the files are deterministic. 27 # TODO(sebmarchand): Remove this once all the files are deterministic.
28 WHITELIST = { 28 WHITELIST = {
29 # https://crbug.com/383340 29 # https://crbug.com/383340
30 'android': { 30 'android': {
31 'flatc', 31 'flatc',
32 }, 32 },
33 33
34 # https://crbug.com/330263 34 # https://crbug.com/330263
35 'linux': { 35 'linux': {
36 'flatc', 36 'flatc',
37 # https://crbug.com/654989
38 'ppapi_nacl_tests_pnacl_newlib_x32_nonsfi.nexe',
39 'ppapi_nacl_tests_pnacl_newlib_x64.nexe',
37 }, 40 },
38 41
39 # https://crbug.com/330262 42 # https://crbug.com/330262
40 'mac': { 43 'mac': {
41 'accelerated_widget_mac_unittests', 44 'accelerated_widget_mac_unittests',
42 'accessibility_unittests', 45 'accessibility_unittests',
43 'angle_end2end_tests', 46 'angle_end2end_tests',
44 'angle_unittests', 47 'angle_unittests',
45 'App Shell', 48 'App Shell',
46 'app_shell_unittests', 49 'app_shell_unittests',
(...skipping 641 matching lines...) Expand 10 before | Expand all | Expand 10 after
688 parser.error('--target-platform is required') 691 parser.error('--target-platform is required')
689 692
690 return compare_build_artifacts(os.path.abspath(options.first_build_dir), 693 return compare_build_artifacts(os.path.abspath(options.first_build_dir),
691 os.path.abspath(options.second_build_dir), 694 os.path.abspath(options.second_build_dir),
692 options.target_platform, 695 options.target_platform,
693 options.recursive) 696 options.recursive)
694 697
695 698
696 if __name__ == '__main__': 699 if __name__ == '__main__':
697 sys.exit(main()) 700 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698