summaryrefslogtreecommitdiffstats
path: root/vndk
diff options
context:
space:
mode:
authorLogan Chien <loganchien@google.com>2019-02-20 16:42:28 +0800
committerLogan Chien <loganchien@google.com>2019-02-27 22:45:45 +0800
commit7d382da66ea1a4a5cb624a81d3700e4e0ce17f4e (patch)
tree95894b5617480ba5680448e0bf72db9c92972de1 /vndk
parentf91fb912d3f5dce25c4df76503435dd275016a00 (diff)
downloadandroid_development-7d382da66ea1a4a5cb624a81d3700e4e0ce17f4e.tar.gz
android_development-7d382da66ea1a4a5cb624a81d3700e4e0ce17f4e.tar.bz2
android_development-7d382da66ea1a4a5cb624a81d3700e4e0ce17f4e.zip
Remove workaround to find the latest lsdump files
This commit removes the workaround to find the latest lsdump files since aosp/858259 has been merged for a while. Bug: 121986692 Test: n/a Change-Id: I1508aefd178e014291cc8f409fdccf527588335a
Diffstat (limited to 'vndk')
-rwxr-xr-xvndk/tools/header-checker/utils/create_reference_dumps.py41
1 files changed, 2 insertions, 39 deletions
diff --git a/vndk/tools/header-checker/utils/create_reference_dumps.py b/vndk/tools/header-checker/utils/create_reference_dumps.py
index 80e51d232..e25ec6135 100755
--- a/vndk/tools/header-checker/utils/create_reference_dumps.py
+++ b/vndk/tools/header-checker/utils/create_reference_dumps.py
@@ -65,39 +65,6 @@ def get_lib_arch_str(target):
return target.arch + target_arch_variant_str
-# FIXME (b/121986692): Before aosp/858259 is merged, apex mutator adds
-# `_platform` suffix if the module is used by some apex modules. This
-# workaround searches for lsdump with and without `_platform`.
-_WORKAROUND_APEX_PLATFORM_LSDUMPS = True
-
-if _WORKAROUND_APEX_PLATFORM_LSDUMPS:
- def _find_lib_lsdumps_workaround(dir_name, lsdump_paths, libs):
- dir_name_platform = dir_name + '_platform'
-
- matched_lsdump_paths = set(find_lib_lsdumps(
- dir_name, lsdump_paths, libs))
-
- matched_lsdump_paths_platform = set(find_lib_lsdumps(
- dir_name_platform, lsdump_paths, libs))
-
- # Pick the lsdump file with latest modification time if both of them
- # exist.
- matched_lsdump_paths_replaced = set(
- path.replace(dir_name, dir_name_platform)
- for path in matched_lsdump_paths)
-
- both = matched_lsdump_paths_replaced & matched_lsdump_paths_platform
-
- for path_platform in both:
- path = path_platform.replace(dir_name_platform, dir_name)
- if os.stat(path).st_mtime >= os.stat(path_platform).st_mtime:
- matched_lsdump_paths_platform.remove(path_platform)
- else:
- matched_lsdump_paths.remove(path)
-
- return sorted(matched_lsdump_paths | matched_lsdump_paths_platform)
-
-
def find_and_copy_lib_lsdumps(target, ref_dump_dir_stem, ref_dump_dir_insertion,
core_or_vendor_shared_str, libs, lsdump_paths,
compress):
@@ -105,12 +72,8 @@ def find_and_copy_lib_lsdumps(target, ref_dump_dir_stem, ref_dump_dir_insertion,
target.arch, target.arch_variant, target.cpu_variant,
core_or_vendor_shared_str)
- if _WORKAROUND_APEX_PLATFORM_LSDUMPS:
- arch_lsdump_paths = _find_lib_lsdumps_workaround(
- module_variant_dir_name, lsdump_paths, libs)
- else:
- arch_lsdump_paths = find_lib_lsdumps(
- module_variant_dir_name, lsdump_paths, libs)
+ arch_lsdump_paths = find_lib_lsdumps(
+ module_variant_dir_name, lsdump_paths, libs)
# Copy the contents of the lsdump into their corresponding reference ABI
# dumps directories.