unit-test: Remove `UBSAN_OPTIONS=halt_on_error=1`
Setting `UBSAN_OPTIONS=halt_on_error=1` here makes adding the
`UBSAN_OPTION` environment variable ineffective (like
`suppressions=ubsan.supp` or `print_summary=1`). This environment
variable is always overwritten.
The mechanism that makes UBSAN kill tests is no longer needed when using
meson version 1.3.0 or higher
https://github.com/mesonbuild/meson/commit/7b7d2e060b where default
```
UBSAN_OPTION=halt_on_error=1:abort_on_error=1:print_summary=1:print_stacktrace=1
```
Resolves: openbmc/openbmc-build-scripts#43
Tested:
- Run `run-unit-test-docker.sh` without setting env variable, with
`invalid vptr` error - tests failed with more info in logs (stack trace,
summary)
- Run `run-unit-test-docker.sh` with setting env variable
`UBSAN_OPTION=suppressions=ubsan.supp` with suppression for vptr check
and with bug location - tests passed
Change-Id: I64eb799645dbc9cde4b7ee75098b641ebe30a5a4
Signed-off-by: Ewelina Walkusz <ewelinax.walkusz@intel.com>
diff --git a/scripts/unit-test.py b/scripts/unit-test.py
index 0dbb403..cd28e21 100755
--- a/scripts/unit-test.py
+++ b/scripts/unit-test.py
@@ -1100,7 +1100,6 @@
"--print-errorlogs",
"--logbase",
"testlog-ubasan",
- env=os.environ | {"UBSAN_OPTIONS": "halt_on_error=1"},
)
# TODO: Fix memory sanitizer
# check_call_cmd('meson', 'configure', 'build',