Re: [dim PATCH 1/7] completion: shellcheck bash completion and fix issues

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

 



On Fri, Mar 24, 2017 at 06:11:53PM +0200, Jani Nikula wrote:
> 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)"

Let's hope this didn't eat errors in some obscure corner case and now
another dim command silently fails :-)

Acked-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
>  
>  	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

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
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