gn2bp: properly forward jni_headers dependency
Gets rid of a hack that used to look at the presence of --sysroot and
blindly added a jni_headers dependency.
Test: //base:base
Change-Id: I629dcf2725f70c7beb87735086f54e01b7d63b86
diff --git a/tools/gn2bp/gen_android_bp b/tools/gn2bp/gen_android_bp
index d01c5bd..2087554 100755
--- a/tools/gn2bp/gen_android_bp
+++ b/tools/gn2bp/gen_android_bp
@@ -685,6 +685,9 @@
break
elif target.script == '//base/android/jni_generator/jni_generator.py':
+ # chromium builds against a prebuilt ndk that contains the jni_headers, so
+ # a dependency is never explicitly created.
+ module.genrule_header_libs.add('jni_headers')
for i, val in enumerate(target.args):
if val == '--output_dir':
# replace --output_dir gen/... with --output_dir $(genDir)/...
@@ -823,12 +826,8 @@
if target.type in gn_utils.LINKER_UNIT_TYPES:
module.cflags.update(_get_cflags(target))
- # HACK! We may have to link against chromium's sysroot instead, but this
- # seems to work for //base:base.
# TODO: implement proper cflag parsing.
for flag in target.cflags:
- if '--sysroot=' in flag:
- module.header_libs.add('jni_headers')
if '-std=' in flag:
module.cpp_std = flag[len('-std='):]