| OLD | NEW |
| 1 #!/usr/bin/env python | 1 #!/usr/bin/env python |
| 2 # | 2 # |
| 3 # Copyright 2013 The Chromium Authors. All rights reserved. | 3 # Copyright 2013 The Chromium Authors. All rights reserved. |
| 4 # Use of this source code is governed by a BSD-style license that can be | 4 # Use of this source code is governed by a BSD-style license that can be |
| 5 # found in the LICENSE file. | 5 # found in the LICENSE file. |
| 6 | 6 |
| 7 import optparse | 7 import optparse |
| 8 import os | 8 import os |
| 9 import shutil | 9 import shutil |
| 10 import re | 10 import re |
| (...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 221 if os.path.exists(options.jar_path): | 221 if os.path.exists(options.jar_path): |
| 222 _ExtractClassFiles(options.jar_path, classes_dir, changed_java_files) | 222 _ExtractClassFiles(options.jar_path, classes_dir, changed_java_files) |
| 223 if os.path.exists(excluded_jar_path): | 223 if os.path.exists(excluded_jar_path): |
| 224 _ExtractClassFiles(excluded_jar_path, classes_dir, changed_java_files) | 224 _ExtractClassFiles(excluded_jar_path, classes_dir, changed_java_files) |
| 225 # Add the extracted files to the classpath. This is required because | 225 # Add the extracted files to the classpath. This is required because |
| 226 # when compiling only a subset of files, classes that haven't changed | 226 # when compiling only a subset of files, classes that haven't changed |
| 227 # need to be findable. | 227 # need to be findable. |
| 228 classpath_idx = javac_cmd.index('-classpath') | 228 classpath_idx = javac_cmd.index('-classpath') |
| 229 javac_cmd[classpath_idx + 1] += ':' + classes_dir | 229 javac_cmd[classpath_idx + 1] += ':' + classes_dir |
| 230 | 230 |
| 231 # Can happen when a target goes from having no sources, to having sources. |
| 232 # It's created by the call to build_utils.Touch() below. |
| 233 if options.incremental: |
| 234 if os.path.exists(pdb_path) and not os.path.getsize(pdb_path): |
| 235 os.unlink(pdb_path) |
| 236 |
| 231 # Don't include the output directory in the initial set of args since it | 237 # Don't include the output directory in the initial set of args since it |
| 232 # being in a temp dir makes it unstable (breaks md5 stamping). | 238 # being in a temp dir makes it unstable (breaks md5 stamping). |
| 233 cmd = javac_cmd + ['-d', classes_dir] + java_files | 239 cmd = javac_cmd + ['-d', classes_dir] + java_files |
| 234 | 240 |
| 235 # JMake prints out some diagnostic logs that we want to ignore. | 241 # JMake prints out some diagnostic logs that we want to ignore. |
| 236 # This assumes that all compiler output goes through stderr. | 242 # This assumes that all compiler output goes through stderr. |
| 237 stdout_filter = lambda s: '' | 243 stdout_filter = lambda s: '' |
| 238 if md5_check.PRINT_EXPLANATIONS: | 244 if md5_check.PRINT_EXPLANATIONS: |
| 239 stdout_filter = None | 245 stdout_filter = None |
| 240 | 246 |
| 241 attempt_build = lambda: build_utils.CheckOutput( | 247 attempt_build = lambda: build_utils.CheckOutput( |
| 242 cmd, | 248 cmd, |
| 243 print_stdout=options.chromium_code, | 249 print_stdout=options.chromium_code, |
| 244 stdout_filter=stdout_filter, | 250 stdout_filter=stdout_filter, |
| 245 stderr_filter=ColorJavacOutput) | 251 stderr_filter=ColorJavacOutput) |
| 246 try: | 252 try: |
| 247 attempt_build() | 253 attempt_build() |
| 248 except build_utils.CalledProcessError as e: | 254 except build_utils.CalledProcessError as e: |
| 249 # Work-around for a bug in jmake (http://crbug.com/551449). | 255 # Work-around for a bug in jmake (http://crbug.com/551449). |
| 250 if 'project database corrupted' not in e.output: | 256 if 'project database corrupted' not in e.output: |
| 251 raise | 257 raise |
| 252 print ('Applying work-around for jmake project database corrupted ' | 258 print ('Applying work-around for jmake project database corrupted ' |
| 253 '(http://crbug.com/551449).') | 259 '(http://crbug.com/551449).') |
| 254 os.unlink(pdb_path) | 260 os.unlink(pdb_path) |
| 255 attempt_build() | 261 attempt_build() |
| 262 elif options.incremental: |
| 263 # Make sure output exists. |
| 264 build_utils.Touch(pdb_path) |
| 256 | 265 |
| 257 if options.main_class or options.manifest_entry: | 266 if options.main_class or options.manifest_entry: |
| 258 entries = [] | 267 entries = [] |
| 259 if options.manifest_entry: | 268 if options.manifest_entry: |
| 260 entries = [e.split(':') for e in options.manifest_entry] | 269 entries = [e.split(':') for e in options.manifest_entry] |
| 261 manifest_file = os.path.join(temp_dir, 'manifest') | 270 manifest_file = os.path.join(temp_dir, 'manifest') |
| 262 _CreateManifest(manifest_file, runtime_classpath, options.main_class, | 271 _CreateManifest(manifest_file, runtime_classpath, options.main_class, |
| 263 entries) | 272 entries) |
| 264 else: | 273 else: |
| 265 manifest_file = None | 274 manifest_file = None |
| (...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 448 options, | 457 options, |
| 449 input_paths=input_paths, | 458 input_paths=input_paths, |
| 450 input_strings=javac_cmd, | 459 input_strings=javac_cmd, |
| 451 output_paths=output_paths, | 460 output_paths=output_paths, |
| 452 force=force, | 461 force=force, |
| 453 pass_changes=True) | 462 pass_changes=True) |
| 454 | 463 |
| 455 | 464 |
| 456 if __name__ == '__main__': | 465 if __name__ == '__main__': |
| 457 sys.exit(main(sys.argv[1:])) | 466 sys.exit(main(sys.argv[1:])) |
| OLD | NEW |