Chromium Code Reviews| Index: third_party/errorprone/errorprone.gyp |
| diff --git a/third_party/errorprone/errorprone.gyp b/third_party/errorprone/errorprone.gyp |
| index 6a1c5db7e73149cb99197526d5d7bbae58163f03..1dbc218f269f7e4d9380973bccde592dd3cc7e16 100644 |
| --- a/third_party/errorprone/errorprone.gyp |
| +++ b/third_party/errorprone/errorprone.gyp |
| @@ -3,13 +3,49 @@ |
| # found in the LICENSE file. |
| { |
| + 'variables': { |
| + 'errorprone_script_path': '<(PRODUCT_DIR)/bin.java/chromium_errorprone', |
| + }, |
| 'targets': [ |
| { |
| - # GN: //third_party/errorprone:errorprone_java |
| - 'target_name': 'errorprone_java', |
| + # GN: //third_party/errorprone:error_prone_annotation_java |
| + 'target_name': 'error_prone_annotation_jar', |
| 'type': 'none', |
| 'variables': { |
| - 'jar_path': 'lib/error_prone_core-1.1.2.jar', |
| + 'jar_path': 'lib/error_prone_annotation-2.0.1.jar', |
| + }, |
| + 'includes': [ |
| + '../../build/host_prebuilt_jar.gypi', |
| + ] |
| + }, |
| + { |
| + # GN: //third_party/errorprone:error_prone_annotations_java |
| + 'target_name': 'error_prone_annotations_jar', |
| + 'type': 'none', |
| + 'variables': { |
| + 'jar_path': 'lib/error_prone_annotations-2.0.1.jar', |
| + }, |
| + 'includes': [ |
| + '../../build/host_prebuilt_jar.gypi', |
| + ] |
| + }, |
| + { |
| + # GN: //third_party/errorprone:javacutil_java |
| + 'target_name': 'javacutil_jar', |
| + 'type': 'none', |
| + 'variables': { |
| + 'jar_path': 'lib/javacutil-1.8.10.jar', |
| + }, |
| + 'includes': [ |
| + '../../build/host_prebuilt_jar.gypi', |
| + ] |
| + }, |
| + { |
| + # GN: //third_party/errorprone:dataflow_java |
| + 'target_name': 'dataflow_jar', |
| + 'type': 'none', |
| + 'variables': { |
| + 'jar_path': 'lib/dataflow-1.8.10.jar', |
| }, |
| 'includes': [ |
| '../../build/host_prebuilt_jar.gypi', |
| @@ -20,17 +56,18 @@ |
| 'target_name': 'chromium_errorprone', |
| 'type': 'none', |
| 'variables': { |
| - 'src_paths': [ |
| - 'src/org/chromium/errorprone/ChromiumErrorProneCompiler.java', |
| - ], |
| - 'enable_errorprone': 0, |
| + 'jar_path': 'lib/error_prone_core-2.0.1.jar', |
| }, |
| 'dependencies': [ |
| '../../build/android/setup.gyp:sun_tools_java', |
| - 'errorprone_java', |
| + '../../third_party/findbugs/findbugs.gyp:format_string_jar', |
| + ':error_prone_annotation_jar', |
|
jbudorick
2015/08/25 19:53:58
nit: while this may work, we don't typically do le
|
| + ':error_prone_annotations_jar', |
| + ':javacutil_jar', |
| + ':dataflow_jar', |
| ], |
| 'includes': [ |
| - '../../build/host_jar.gypi', |
| + '../../build/host_prebuilt_jar.gypi', |
| ], |
| 'actions': [ |
| { |
| @@ -42,14 +79,38 @@ |
| '<(jar_path)', |
| ], |
| 'outputs': [ |
| - '<(PRODUCT_DIR)/bin.java/chromium_errorprone' |
| + '<(errorprone_script_path)', |
| ], |
| 'action': [ |
| 'python', '<(DEPTH)/build/android/gyp/create_java_binary_script.py', |
| '--output', '<(PRODUCT_DIR)/bin.java/chromium_errorprone', |
| - '--classpath=>@(input_jars_paths)', |
| '--jar-path=<(jar_path)', |
| - '--main-class=org.chromium.errorprone.ChromiumErrorProneCompiler', |
| + '--classpath=>@(input_jars_paths)', |
| + '--main-class=com.google.errorprone.ErrorProneCompiler', |
| + ], |
| + }, |
| + ], |
| + }, |
| + { |
| + # This emulates gn's datadeps fields. We don't want the errorprone jars |
| + # to be added to the classpaths of targets that depend on errorprone. |
| + 'target_name': 'require_errorprone', |
| + 'type': 'none', |
| + 'actions': [ |
| + { |
| + 'action_name': 'require_errorprone', |
| + 'message': 'Making sure errorprone has been built.', |
| + 'variables': { |
| + 'required_file': '<(PRODUCT_DIR)/bin.java/errorprone.required', |
| + }, |
| + 'inputs': [ |
| + '<(errorprone_script_path)', |
| + ], |
| + 'outputs': [ |
| + '<(required_file)', |
| + ], |
| + 'action': [ |
| + 'python', '../../build/android/gyp/touch.py', '<(required_file)', |
| ], |
| }, |
| ], |