Chromium Code Reviews| Index: editor/tools/compile_analyzer.py |
| diff --git a/editor/tools/compile_analyzer.py b/editor/tools/compile_analyzer.py |
| index 0a4e3eab17e6f9ddcac2d4093b043fc11815e91b..b6016e0a04a836adc9fc4248595e3e89247f00bb 100644 |
| --- a/editor/tools/compile_analyzer.py |
| +++ b/editor/tools/compile_analyzer.py |
| @@ -35,7 +35,10 @@ def GetOptions(): |
| def CompileAnalyzer(options, args): |
| # We rely on all jar files being copied to the output dir. |
| - class_path = options.output_dir + '*' |
| + if sys.platform == 'win32': |
| + class_path = options.output_dir + '*;' |
| + else: |
| + class_path = options.output_dir + '*' |
| cmd = [GetJavacPath(), |
| '-sourcepath', 'foobar', |
| '-source', '6', |
| @@ -45,7 +48,7 @@ def CompileAnalyzer(options, args): |
| '-cp', class_path, |
| ] |
| cmd.extend(args) |
| - subprocess.call(cmd) |
| + subprocess.check_call(cmd) |
|
ahe
2013/04/12 15:06:49
Unfortunately, we cannot use check_call as it isn'
kustermann
2013/04/12 15:18:08
Done.
|
| def CreateJarFile(options): |
| class_path_file_name = options.output_dir + options.class_path_file |
| @@ -53,7 +56,7 @@ def CreateJarFile(options): |
| cmd = [GetJarToolPath(), 'cfem', jar_file_name, options.entry_point, |
| class_path_file_name, |
| '-C', options.output_dir, options.jar_entry_directory] |
| - subprocess.call(cmd) |
| + subprocess.check_call(cmd) |
| def CopyFiles(options): |
| # Strip " from the string |