diff options
| author | 2022-02-17 21:46:21 -0800 | |
|---|---|---|
| committer | 2022-02-17 21:46:21 -0800 | |
| commit | 07c71ac1b9fc5b36a09a7cd3fe33939e73b684c4 (patch) | |
| tree | 9578c5f93999eaedbbc861f9b12a2e5593709e6c /tools/releasetools/sign_apex.py | |
| parent | 42ab8287546bc07217039643e4e277f1242c17a5 (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-x | tools/releasetools/sign_apex.py | 3 |
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() |