BUG: engine-var must be matched with case and trailing separator and consider quoting in output

inkarkat · inkarkat · commit 2d2e2bd798d3 · 2023-05-24T21:12:56.000+02:00
To avoid that e.g. @Open-S matches @open-searcher-command. If the configured command contains whitespace, Tmux will double-quote the result, and we must pick up all words, not just the first one.
diff --git a/scripts/helpers.sh b/scripts/helpers.sh
@@ -43,7 +43,7 @@ stored_engine_vars() {
 
 get_engine() {
 	local engine_var="$1"
-	tmux show-options -g | grep -i "^@open-$engine_var" | cut -d ' ' -f2 | xargs
+	tmux show-options -g | grep "^@open-$engine_var" | cut -d ' ' -f2- | xargs
 }
 
 # The last grep is required to remove non-digits from version such as "3.0a".

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ stored_engine_vars() {`
`43`	`43`
`44`	`44`	`get_engine() {`
`45`	`45`	`local engine_var="$1"`
`46`		`- tmux show-options -g \| grep -i "^@open-$engine_var" \| cut -d ' ' -f2 \| xargs`
	`46`	`+ tmux show-options -g \| grep "^@open-$engine_var" \| cut -d ' ' -f2- \| xargs`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`# The last grep is required to remove non-digits from version such as "3.0a".`