summaryrefslogtreecommitdiff
path: root/tools/releasetools/sign_apex.py
diff options
context:
space:
mode:
author Kelvin Zhang <zhangkelvin@google.com> 2022-02-17 21:46:21 -0800
committer Kelvin Zhang <zhangkelvin@google.com> 2022-02-17 21:46:21 -0800
commit07c71ac1b9fc5b36a09a7cd3fe33939e73b684c4 (patch)
tree9578c5f93999eaedbbc861f9b12a2e5593709e6c /tools/releasetools/sign_apex.py
parent42ab8287546bc07217039643e4e277f1242c17a5 (diff)
Remove catch + sys.exit idiom
This is bad, python already prints exceptino message and exit with non-zero code when an exception is raised, no need to catch the exception just to print it. In addition, printing an exception does not print stacktrace, so catch + sys.exit provide less information than python's default behavior. Test: th Change-Id: If6cf2e34d4ebc9f7d172063b4396bf4377dad447
Diffstat (limited to 'tools/releasetools/sign_apex.py')
-rwxr-xr-xtools/releasetools/sign_apex.py3
1 files changed, 0 insertions, 3 deletions
diff --git a/tools/releasetools/sign_apex.py b/tools/releasetools/sign_apex.py
index 66f5e0513a..692646705e 100755
--- a/tools/releasetools/sign_apex.py
+++ b/tools/releasetools/sign_apex.py
@@ -149,8 +149,5 @@ def main(argv):
if __name__ == '__main__':
try:
main(sys.argv[1:])
- except common.ExternalError:
- logger.exception("\n ERROR:\n")
- sys.exit(1)
finally:
common.Cleanup()