Merge "Fix manifest_check.py for generated manifests that have no application tag" into main

This commit is contained in:
Treehugger Robot
2023-10-13 22:34:46 +00:00
committed by Gerrit Code Review

View File

@@ -187,18 +187,17 @@ def extract_uses_libs_apk(badging):
return required, optional, tags return required, optional, tags
def extract_uses_libs_xml(xml): #pylint: disable=inconsistent-return-statements def extract_uses_libs_xml(xml):
"""Extract <uses-library> tags from the manifest.""" """Extract <uses-library> tags from the manifest."""
manifest = parse_manifest(xml) manifest = parse_manifest(xml)
elems = get_children_with_tag(manifest, 'application') elems = get_children_with_tag(manifest, 'application')
application = elems[0] if len(elems) == 1 else None if len(elems) > 1:
if len(elems) > 1: #pylint: disable=no-else-raise
raise RuntimeError('found multiple <application> tags') raise RuntimeError('found multiple <application> tags')
elif not elems: if not elems:
if uses_libraries or optional_uses_libraries: #pylint: disable=undefined-variable return [], [], []
raise ManifestMismatchError('no <application> tag found')
return application = elems[0]
libs = get_children_with_tag(application, 'uses-library') libs = get_children_with_tag(application, 'uses-library')