summaryrefslogtreecommitdiff
path: root/scripts/lint_project_xml.py
diff options
context:
space:
mode:
author Cole Faust <colefaust@google.com> 2024-01-17 17:11:51 -0800
committer Cole Faust <colefaust@google.com> 2024-01-17 17:11:51 -0800
commit9809860ea82134ef79b7d9d6415192b3ee76ccc4 (patch)
treedbfcce57c933f4b10c2143c1b61e72f0c2d11bdf /scripts/lint_project_xml.py
parent6794cc20c55e30a9f83871cfb9dee142ec5aac3c (diff)
Simplify strict updatability linting error message
Throwing a RuntimeError leads to a stack trace in the build failure error message, which is not useful and makes the error harder to understand. Bug: 320698986 Test: Presubmits Change-Id: I0b79185fff7b9c8d4085b23936b80b3d026a4384
Diffstat (limited to 'scripts/lint_project_xml.py')
-rwxr-xr-xscripts/lint_project_xml.py5
1 files changed, 3 insertions, 2 deletions
diff --git a/scripts/lint_project_xml.py b/scripts/lint_project_xml.py
index 3b0158d0b..c40b07d38 100755
--- a/scripts/lint_project_xml.py
+++ b/scripts/lint_project_xml.py
@@ -18,6 +18,7 @@
"""This file generates project.xml and lint.xml files used to drive the Android Lint CLI tool."""
import argparse
+import sys
from xml.dom import minidom
from ninja_rsp import NinjaRspFileReader
@@ -159,8 +160,8 @@ def main():
if args.baseline_path:
baseline = minidom.parse(args.baseline_path)
disallowed_issues = check_baseline_for_disallowed_issues(baseline, args.disallowed_issues)
- if bool(disallowed_issues):
- raise RuntimeError('disallowed issues %s found in lint baseline file %s for module %s'
+ if disallowed_issues:
+ sys.exit('disallowed issues %s found in lint baseline file %s for module %s'
% (disallowed_issues, args.baseline_path, args.name))
if args.project_out: