diff options
| author | 2025-03-11 09:33:06 -0700 | |
|---|---|---|
| committer | 2025-03-11 09:41:16 -0700 | |
| commit | 012cfd094c7de9203ef801745ba4ae45eb036196 (patch) | |
| tree | f1d9ec5a8cfb0ea73c5daecdfeb423953c4cf826 | |
| parent | a369dc00143821fa7784d1bfa9d2986186b23742 (diff) | |
manifest_check.py: Clarify that ordering matters
When comparing the library declaration from AndroidManifest.xml and
Android.bp, the library ordering must match (see b/132357300#comment3).
The error message does not make it clear that ordering is significant
and a mismatch in ordering is an error.
Improve the error message to make this more obvious.
Bug: 401287317
Test: compiles
Change-Id: Idfb3e1e96893f81190dc320c49d40677b76f9853
| -rwxr-xr-x | scripts/manifest_check.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/manifest_check.py b/scripts/manifest_check.py index 175451e74..96dc5a6ce 100755 --- a/scripts/manifest_check.py +++ b/scripts/manifest_check.py @@ -132,7 +132,7 @@ def enforce_uses_libraries(manifest, required, optional, missing_optional, relax #pylint: disable=line-too-long errmsg = ''.join([ - 'mismatch in the <uses-library> tags between the build system and the ' + 'mismatch or misordering in the <uses-library> tags between the build system and the ' 'manifest:\n', '\t- required libraries in build system: %s[%s]%s\n' % (C_RED, ', '.join(required), C_OFF), '\t vs. in the manifest: %s[%s]%s\n' % (C_RED, ', '.join(manifest_required), C_OFF), |