[dim PATCH] completion: shellcheck bash completion and fix issues

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Add bash_completion to 'make shellcheck' target, and fix the failures.

Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
---
 Makefile        |  2 +-
 bash_completion | 13 +++++++------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/Makefile b/Makefile
index 30fc3dd3bff8..c1fe02c1b565 100644
--- a/Makefile
+++ b/Makefile
@@ -30,7 +30,7 @@ SC_EXCLUDE := \
 	-e SC2143
 
 shellcheck:
-	shellcheck $(SC_EXCLUDE) dim
+	shellcheck $(SC_EXCLUDE) dim bash_completion
 
 clean:
 	rm -f drm-intel.html drm-intel-flow.svg dim.html drm-misc.html
diff --git a/bash_completion b/bash_completion
index 9f659b4ebcce..7dfc4b86cb13 100644
--- a/bash_completion
+++ b/bash_completion
@@ -3,7 +3,7 @@
 dim ()
 {
 	if [[ "x$1" = "xcd" ]]; then
-		cd `cat ~/.dim-last-path`
+		cd $(cat ~/.dim-last-path) || exit
 	else
 		command dim "$@"
 	fi
@@ -12,6 +12,7 @@ dim ()
 _dim ()
 {
 	local args arg cur prev words cword split
+	local nightly_branches upstream_branches opts cmds aliasref
 
 	# require bash-completion with _init_completion
 	type -t _init_completion >/dev/null 2>&1 || return
@@ -26,18 +27,18 @@ _dim ()
 	# args = number of arguments
 	_count_args
 
-	local nightly_branches="$(dim list-branches)"
-	local upstream_branches="$(dim list-upstreams)"
+	nightly_branches="$(dim list-branches)"
+	upstream_branches="$(dim list-upstreams)"
 
 	if [ -z "${arg}" ]; then
 		# top level completion
 		case "${cur}" in
 			-*)
-				local opts="-d -f -i"
+				opts="-d -f -i"
 				COMPREPLY=( $(compgen -W "${opts}" -- ${cur}) )
 				;;
 			*)
-				local cmds="$(dim list-commands) $(dim list-aliases | sed 's/\t.*//')"
+				cmds="$(dim list-commands) $(dim list-aliases | sed 's/\t.*//')"
 				COMPREPLY=( $(compgen -W "${cmds}" -- ${cur}) )
 				;;
 		esac
@@ -45,7 +46,7 @@ _dim ()
 	fi
 
 	# complete aliases like the actual command
-	local aliasref=$(dim list-aliases | sed -n "s/^${arg}\t\(.*\)/\1/p")
+	aliasref=$(dim list-aliases | sed -n "s/^${arg}\t\(.*\)/\1/p")
 	if [[ -n "$aliasref" ]]; then
 		arg="$aliasref"
 	fi
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux