Merge pull request #86 from mlawson-tt/fix-run-return-code-no-browser

Ensure a non-zero return code is set if expected when --no-browser is provided.
riscv-software-src · Aug 24, 2023 · a2b4cd7 · a2b4cd7
2 parents eac7c9d + 222d11b
commit a2b4cd7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@ Only when a release to the main branch is done, the contents of the WIP-DEV are
 versioned header while the `WIP-DEV` is left empty
 
 ## [WIP-DEV]
+- Ensure correct riscof return code when `--no-browser` is used. Fixes #87.
 - Updating CONTRIBUTING.rst to capture the new git strategy adopted to follow a monthly release
 
 ## [1.25.3] - 2023-01-24

diff --git a/riscof/cli.py b/riscof/cli.py
@@ -352,6 +352,8 @@ def run(ctx,config,work_dir,suite,env,no_browser,dbfile,testfile,no_ref_run,no_d
             raise SystemExit(exitcode)
         except:
             raise SystemExit(exitcode)
+    elif exitcode:
+        raise SystemExit(exitcode)