shellcheck was run on perf tool shell scripts as a pre-requisite to include a build option for shellcheck discussed here: https://www.spinics.net/lists/linux-perf-users/msg25553.html
And fixes were added for the coding/formatting issues in two patchsets: https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atraj...@linux.vnet.ibm.com/ https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atraj...@linux.vnet.ibm.com/ Three additional issues were observed and fixes are part of: git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git With recent commits in perf, other three issues are observed. shellcheck version: 0.6.0 The changes are with recent commits ( which is mentioned in each patch) for lock_contention, record_sideband and stat_all_metricgroups test. Patch series fixes these testcases and patches are on top of: git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git The version 1 patchset had fix patch for test_arm_coresight.sh. Dropping that in V2 based on discussion here: https://lore.kernel.org/linux-perf-users/f265857d-0d37-4878-908c-d20732f21...@linux.vnet.ibm.com/T/#u Athira Rajeev (3): tools/perf/tests Ignore the shellcheck SC2046 warning in lock_contention tools/perf/tests: Fix shellcheck warning in record_sideband.sh test tools/perf/tests/shell: Fix shellcheck warning SC2112 with stat_all_metricgroups tools/perf/tests/shell/lock_contention.sh | 1 + tools/perf/tests/shell/record_sideband.sh | 2 +- tools/perf/tests/shell/stat_all_metricgroups.sh | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) -- 2.31.1
