[Intel-gfx] [maintainer-tools PATCH 4/4] dim: fix list-aliases to not include itself

Rodrigo Vivi rodrigo.vivi at gmail.com
Fri Aug 11 17:57:01 UTC 2017


On Fri, Aug 11, 2017 at 06:03:25PM +0300, Jani Nikula wrote:
> dim list-aliases lists "list-aliases" as being an alias of
> "list-aliases". This is because the temporary subcmd variable reference
> is erroneously included in the list. Unset it.

I got confused with to many list-aliases list-aliases, so I run dim list-aliases
and understood what you meant... ;)


Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>

> 
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
>  dim | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/dim b/dim
> index d9820ac87358..011cedbd3b6a 100755
> --- a/dim
> +++ b/dim
> @@ -2053,4 +2053,7 @@ if ! declare -f $subcmd_func >/dev/null; then
>  	exit 1
>  fi
>  
> +# throw away to not confuse list-aliases
> +unset subcmd
> +
>  $subcmd_func "$@"
> -- 
> 2.11.0
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


More information about the Intel-gfx mailing list