Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 Loading... | |
| 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 Loading... | |
| 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 Loading... | |
| 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()) |
| OLD | NEW |