Skip to content

Commit

Permalink
Print summary table even when some proofs fail
Browse files Browse the repository at this point in the history
Prior to this commit, passing --fail-on-pipeline-failure would terminate
run-cbmc-proofs.py without printing the summary table if some of the
proofs failed. This commit fixes that.
  • Loading branch information
karkhaz authored and markrtuttle committed Jul 11, 2022
1 parent 1a76b72 commit 41e8fc7
Showing 1 changed file with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -219,17 +219,18 @@ def run_build(litani, jobs, fail_on_proof_failure, summarize):

logging.debug(" ".join(cmd))
proc = subprocess.run(cmd, check=False)
if proc.returncode:
if fail_on_proof_failure:
logging.error("One or more proofs failed")
sys.exit(10)

if proc.returncode and not fail_on_proof_failure:
logging.critical("Failed to run litani run-build")
sys.exit(1)

if summarize:
print_proof_results(out_file)
out_file.unlink()

if proc.returncode:
logging.error("One or more proofs failed")
sys.exit(10)

def get_litani_path(proof_root):
cmd = [
Expand Down

0 comments on commit 41e8fc7

Please sign in to comment.