Commit c6043fe
tools: fix get_asan_state() in tools/test.py
The output of `node -p process.config.variables.asan` includes
a newline character so it's never exactly "1", which means
asan is always "off" for the status files. This fixes the
detection by stripping whitespaces from the output.
PR-URL: #52766
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>1 parent 6e71acc commit c6043fe
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1613,7 +1613,7 @@ def get_env_type(vm, options_type, context): | |||
| 1613 | 1613 | ||
| 1614 | 1614 | ||
| 1615 | 1615 | def get_asan_state(vm, context): | |
| 1616 | - asan = Execute([vm, '-p', 'process.config.variables.asan'], context).stdout | ||
| 1616 | + asan = Execute([vm, '-p', 'process.config.variables.asan'], context).stdout.strip() | ||
| 1617 | 1617 | return "on" if asan == "1" else "off" | |
| 1618 | 1618 | ||
| 1619 | 1619 | ||
0 commit comments