[PATCH] dim: don't opencode fetch_all in dim_update_branches

Daniel Vetter daniel at ffwll.ch
Wed Oct 24 08:04:30 UTC 2018


On Tue, Oct 23, 2018 at 04:32:47PM +0100, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> Simply make use of the function, instead of duplicating it.
> 
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>

Applied, thanks.
-Daniel

> ---
>  dim | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/dim b/dim
> index 8be504e46995..21e1e0d6ed45 100755
> --- a/dim
> +++ b/dim
> @@ -2026,12 +2026,7 @@ function dim_update_branches
>  
>  	cd $DIM_PREFIX/$DIM_REPO
>  
> -	for repo in "${!drm_tip_repos[@]}"; do
> -		remote=$(repo_to_remote $repo)
> -		echo -n "Fetching $repo (local remote $remote)... "
> -		git_fetch_helper $remote
> -		echo "Done."
> -	done
> +	fetch_all
>  
>  	assert_repo_clean
>  
> -- 
> 2.19.1
> 
> _______________________________________________
> 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


More information about the dim-tools mailing list