Skip to content

Commit bd83ea8

Browse files
committed
Better fpga-tool-perf error handling
Signed-off-by: Maciej Kurc <[email protected]>
1 parent ce64c61 commit bd83ea8

File tree

1 file changed

+13
-2
lines changed

1 file changed

+13
-2
lines changed

fpgaperf.py

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,13 @@ def run(
271271
err = str(e)
272272
if not verbose and len(err) > 1000:
273273
err = f"[...]\n{err[-1000:]}"
274-
logger.debug(f"ERROR: {err}")
274+
logger.error(err)
275+
276+
if verbose:
277+
trace = traceback.format_exc()
278+
for line in trace.split("\n"):
279+
logger.error(line)
280+
275281
err = err.split("\n")
276282
else:
277283
logger.debug("Printing Stats")
@@ -280,6 +286,8 @@ def run(
280286
logger.debug("Writing Metadata")
281287
tch.write_metadata(err)
282288

289+
return (err is None)
290+
283291

284292
def get_combinations():
285293
""" Returns a list of tuples with all the possible combinations of supported builds """
@@ -628,7 +636,7 @@ def main():
628636
sys.exit(1)
629637

630638
seed = int(args.seed, 0) if args.seed else None
631-
run(
639+
res = run(
632640
args.board,
633641
args.toolchain,
634642
args.project,
@@ -645,6 +653,9 @@ def main():
645653
build_type=args.build_type
646654
)
647655

656+
if res == False:
657+
exit(-1)
658+
648659

649660
if __name__ == '__main__':
650661
main()

0 commit comments

Comments
 (0)