Merge pull request ClickHouse#79544 from azat/tests/bash-version-check

nikitamikhaylov · web-flow · commit f2d2faa0dd4d · 2025-04-24T14:45:40.000Z
tests: enable debugging via BASH_XTRACEFD only if bash is new enough
diff --git a/tests/queries/shell_config.sh b/tests/queries/shell_config.sh
@@ -214,7 +214,8 @@ function run_with_error()
     return 0
 }
 
-if [[ -n $CLICKHOUSE_BASH_TRACING_FILE ]]; then
+# BASH_XTRACEFD is supported only since 4.1
+if [[ -n $CLICKHOUSE_BASH_TRACING_FILE ]] && [[ ${BASH_VERSINFO[0]} -gt 4 || (${BASH_VERSINFO[0]} -eq 4 && ${BASH_VERSINFO[1]} -ge 1) ]]; then
     exec 3>"$CLICKHOUSE_BASH_TRACING_FILE"
     # It will be also nice to have stderr in the tracing output, but:
     # - exec 2>&3

Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,8 @@ function run_with_error()`
`214`	`214`	`return 0`
`215`	`215`	`}`
`216`	`216`
`217`		`-if [[ -n $CLICKHOUSE_BASH_TRACING_FILE ]]; then`
	`217`	`+# BASH_XTRACEFD is supported only since 4.1`
	`218`	`+if [[ -n $CLICKHOUSE_BASH_TRACING_FILE ]] && [[ ${BASH_VERSINFO[0]} -gt 4 \|\| (${BASH_VERSINFO[0]} -eq 4 && ${BASH_VERSINFO[1]} -ge 1) ]]; then`
`218`	`219`	`exec 3>"$CLICKHOUSE_BASH_TRACING_FILE"`
`219`	`220`	`# It will be also nice to have stderr in the tracing output, but:`
`220`	`221`	`# - exec 2>&3`