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

Side by Side Diff: scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

Issue 1086193002: Remove some files from the Windows non-deterministic whitelist. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years, 8 months 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 12 matching lines...) Expand all
23 # 23 #
24 # PNaCl general bug: http://crbug.com/429358 24 # PNaCl general bug: http://crbug.com/429358
25 # 25 #
26 # TODO(sebmarchand): Remove this once all the files are deterministic. 26 # TODO(sebmarchand): Remove this once all the files are deterministic.
27 WHITELIST = freeze({ 27 WHITELIST = freeze({
28 # http://crbug.com/383340 28 # http://crbug.com/383340
29 'android': { 29 'android': {
30 # Completed. 30 # Completed.
31 }, 31 },
32 32
33 # http://crbug.com/383364
M-A Ruel 2015/04/15 20:44:46 Please refer this bug too in the CL description to
34 'ios': {
35 # TODO(maruel): Add files.
36 },
37
38 # http://crbug.com/330263 33 # http://crbug.com/330263
39 'linux': { 34 'linux': {
40 'browser_tests.isolated', 35 'browser_tests.isolated',
41 'irt_exception_test_pnacl_newlib_x64.nexe', 36 'irt_exception_test_pnacl_newlib_x64.nexe',
42 'irt_manifest_file_pnacl_newlib_x64.nexe', 37 'irt_manifest_file_pnacl_newlib_x64.nexe',
43 'ppapi_tests_extensions_packaged_app_pnacl_newlib_x64.nexe', 38 'ppapi_tests_extensions_packaged_app_pnacl_newlib_x64.nexe',
44 }, 39 },
45 40
46 # http://crbug.com/330262 41 # http://crbug.com/330262
47 'mac': { 42 'mac': {
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 'interactive_ui_tests.exe', 104 'interactive_ui_tests.exe',
110 'interactive_ui_tests.isolated', 105 'interactive_ui_tests.isolated',
111 'metro_driver.dll', 106 'metro_driver.dll',
112 'mksnapshot.exe', 107 'mksnapshot.exe',
113 'mock_nacl_gdb.exe', 108 'mock_nacl_gdb.exe',
114 'net_unittests.exe', 109 'net_unittests.exe',
115 'net_unittests.isolated', 110 'net_unittests.isolated',
116 'np_test_netscape_plugin.dll', 111 'np_test_netscape_plugin.dll',
117 'npapi_test_plugin.dll', 112 'npapi_test_plugin.dll',
118 'peerconnection_server.exe', 113 'peerconnection_server.exe',
119 'ppapi_nacl_tests_pnacl_newlib_x32.nexe',
120 'ppapi_nacl_tests_pnacl_newlib_x64.nexe',
121 'sync_integration_tests.exe', 114 'sync_integration_tests.exe',
122 'sync_integration_tests.isolated', 115 'sync_integration_tests.isolated',
123 'test_registrar.exe', 116 'test_registrar.exe',
124 'unit_tests.exe', 117 'unit_tests.exe',
125 'unit_tests.isolated', 118 'unit_tests.isolated',
126 }, 119 },
127 }) 120 })
128 121
129 def get_files_to_compare(build_dir, recursive=False): 122 def get_files_to_compare(build_dir, recursive=False):
130 """Get the list of files to compare.""" 123 """Get the list of files to compare."""
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 parser.error('--target-platform is required') 314 parser.error('--target-platform is required')
322 315
323 return compare_build_artifacts(os.path.abspath(options.first_build_dir), 316 return compare_build_artifacts(os.path.abspath(options.first_build_dir),
324 os.path.abspath(options.second_build_dir), 317 os.path.abspath(options.second_build_dir),
325 options.target_platform, 318 options.target_platform,
326 options.recursive) 319 options.recursive)
327 320
328 321
329 if __name__ == '__main__': 322 if __name__ == '__main__':
330 sys.exit(main()) 323 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