Messages in this thread Patch in this message | | | Date | Mon, 21 Jun 2021 14:56:46 -0700 | Subject | [PATCH v2 1/3] perf test: Pass the verbose option to shell tests | From | Ian Rogers <> |
| |
Having a verbose option will allow shell tests to provide extra failure details when the fail or skip.
Signed-off-by: Ian Rogers <irogers@google.com> --- tools/perf/tests/builtin-test.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/tools/perf/tests/builtin-test.c b/tools/perf/tests/builtin-test.c index cbbfe48ab802..e1ed60567b2f 100644 --- a/tools/perf/tests/builtin-test.c +++ b/tools/perf/tests/builtin-test.c @@ -577,10 +577,13 @@ struct shell_test { static int shell_test__run(struct test *test, int subdir __maybe_unused) { int err; - char script[PATH_MAX]; + char script[PATH_MAX + 3]; struct shell_test *st = test->priv; - path__join(script, sizeof(script), st->dir, st->file); + path__join(script, sizeof(script) - 3, st->dir, st->file); + + if (verbose) + strncat(script, " -v", sizeof(script) - strlen(script) - 1); err = system(script); if (!err) -- 2.32.0.288.g62a8d224e6-goog
| |