[PATCH 1/9] dim: abstract listing the developer subcommands

[Daniel Vetter]

On Thu, Mar 29, 2018 at 11:45:53AM +0300, Jani Nikula wrote:
> Make the developer subcommand checking less clunky.
> 
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
>  dim | 54 ++++++++++++++++++++++++------------------------------
>  1 file changed, 24 insertions(+), 30 deletions(-)
> 
> diff --git a/dim b/dim
> index 29647f87381e..c32bbddde6be 100755
> --- a/dim
> +++ b/dim
> @@ -2057,6 +2057,25 @@ function dim_list_aliases
>  		sed 's/^dim_alias_//;s/=/\t/;s/_/-/g'
>  }
>  
> +# Commands that do not require full setup
> +function list_developer_commands
> +{
> +	# These should match the developer section in dim.rst, plus help and
> +	# setup commands
> +	cat <<-EOF
> +		checker
> +		checkpatch
> +		cite
> +		fixes
> +		help
> +		retip
> +		setup
> +		sparse
> +		tc
> +		usage

We loose the in-line comments with this, not super stoked about that ...
-Daniel

> +	EOF
> +}
> +
>  function dim_cat_to_fixup # [branch]
>  {
>  	local fixup_file repo branch
> @@ -2294,33 +2313,12 @@ if ! declare -f $subcmd_func >/dev/null; then
>  	exit 1
>  fi
>  
> -# functions useful for developers don't need a full dim setup
> -developer_commands=(
> -	# developer commands
> -	# these should match the developer section in dim.rst
> -	"tc"
> -	"cite"
> -	"fixes"
> -	"checkpatch"
> -	"sparse"
> -	"checker"
> -	"retip"
> -	# help commands
> -	"help"
> -	"usage"
> -	# include setup
> -	"setup"
> -)
> -
> -developer_command=0
> -for cmd in "${developer_commands[@]}" ; do
> -	if [ $cmd = $subcmd ] ; then
> -		developer_command=1
> -		break;
> +# Commands useful for developers don't need a full dim setup
> +if list_developer_commands | grep -qx $subcmd; then
> +	if [ -r $DIM_PREFIX/drm-rerere/$integration_config ]; then
> +		read_integration_config
>  	fi
> -done
> -
> -if [ $developer_command = "0" ]; then
> +else
>  	for d in $DIM_PREFIX $DIM_PREFIX/$DIM_REPO $DIM_PREFIX/drm-rerere $DIM_PREFIX/drm-tip; do
>  		if [ ! -d $d ]; then
>  			echoerr "$d is missing, please check your configuration and/or run dim setup"
> @@ -2329,10 +2327,6 @@ if [ $developer_command = "0" ]; then
>  	done
>  
>  	read_integration_config
> -else
> -	if [ -r $DIM_PREFIX/drm-rerere/$integration_config ]; then
> -		read_integration_config
> -	fi
>  fi
>  
>  # throw away to not confuse list-aliases
> -- 
> 2.11.0
> 
> _______________________________________________
> dim-tools mailing list
> dim-tools at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dim-tools

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch