Remove potentially undefined exit behaviour from analyzesize tool

Python2.7 doc for `sys` module states that "Most systems require [the
exit code] to be in the range 0-127, and produce undefined results
otherwise".

Currently, modern Linux-based systems will exit with code 255, while
msys2-based systems such as Proxspace will exit with code 127.

Instead, explicitly exit with non-zero exit code of 1.
This commit is contained in:
Joe Harrison 2020-02-21 13:37:43 +00:00
commit 788e8b7ecc

View file

@ -17,7 +17,7 @@ except FileNotFoundError:
if len(sys.argv) < 3:
print("Usage: analyzesize.py <info|add|diff> <datasetname>")
exit(-1)
exit(2)
action, name = sys.argv[1:3]
currentdata = subprocess.run(["arm-none-eabi-size","armsrc/obj/fullimage.stage1.elf"], stdout=subprocess.PIPE).stdout
currentdata = currentdata.split(b"\n")[1].strip()