Merge "Fix manifest_check.py for generated manifests that have no application tag" into main
This commit is contained in:
@@ -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')
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user