gn2bp: depend on :current_android_jar for JNI generation

Test: update_results.sh
Change-Id: Id08cb64c65cc2f2b2136d6e437ad8af1876da180
This commit is contained in:
Patrick Rohr 2022-12-22 11:39:35 -08:00
commit 7d2cdf44bc
2 changed files with 7 additions and 4 deletions

View file

@ -318,7 +318,7 @@ cc_genrule {
"--includes " +
"base/android/jni_generator/jni_generator_helper.h " +
"--jar_file " +
"$(location third_party/android_sdk/public/platforms/android-33/android.jar) " +
"$(location :current_android_jar) " +
"--output_name " +
"Runnable_jni.h " +
"--output_name " +
@ -334,12 +334,12 @@ cc_genrule {
"base/android_runtime_jni_headers/Runtime_jni.h",
],
tool_files: [
":current_android_jar",
"base/android/jni_generator/android_jar.classes",
"base/android/jni_generator/jni_generator.py",
"build/android/gyp/util/__init__.py",
"build/android/gyp/util/build_utils.py",
"build/gn_helpers.py",
"third_party/android_sdk/public/platforms/android-33/android.jar",
],
apex_available: [
"com.android.tethering",

View file

@ -924,8 +924,7 @@ class JniGeneratorSanitizer(BaseActionSanitizer):
return arg
def _sanitize_args(self):
self._update_value_arg('--jar_file', self._sanitize_filepath, False)
self._update_value_arg('--jar_file', self._add_location_tag, False)
self._set_value_arg('--jar_file', '$(location :current_android_jar)', False)
if self._has_arg('--jar_file'):
self._append_arg('--javap', '$$(find out/.path -name javap)')
self._update_value_arg('--output_dir', self._sanitize_filepath)
@ -945,6 +944,10 @@ class JniGeneratorSanitizer(BaseActionSanitizer):
# android_jar.classes should be part of the tools as it list implicit classes
# for the script to generate JNI headers.
tool_files.add("base/android/jni_generator/android_jar.classes")
# Filter android.jar and add :current_android_jar
tool_files = {file if not file.endswith('android.jar') else ':current_android_jar'
for file in tool_files }
return tool_files
class JniRegistrationGeneratorSanitizer(BaseActionSanitizer):