On Mon, Oct 07, 2019 at 07:54:02PM +0200, Maxime Ripard wrote:
> All the users of tag_branch do a push for the created tag as the very next
> step. Let's move that into the tag_branch function.
> 
> Signed-off-by: Maxime Ripard <[email protected]>
> ---
>  dim | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/dim b/dim
> index 70262b60d513..eafc1794f6e5 100755
> --- a/dim
> +++ b/dim
> @@ -1957,12 +1957,14 @@ function tag_summary # branch
>  
>  function tag_branch
>  {
> -     local tag branch
> +     local remote tag branch
>       tag=$1
>       branch=$2
> +     remote=$(branch_to_remote $branch)
>  
>       tag_summary $branch | $DRY git tag -F- $tag "$branch@{upstream}"
>       $DRY git tag -a $DIM_GPG_KEYID -f $tag
> +     $DRY git push $remote $tag

This doesn't work, you really need git_push.
-Daniel

>  }
>  
>  # $1: commit subject prefix
> @@ -2010,7 +2012,6 @@ function dim_update_next_continue
>       tag=$(tag_name "drm-intel-next")
>  
>       tag_branch $tag drm-intel-next
> -     git_push $remote $tag
>  }
>  
>  function dim_tag_branch
> @@ -2045,7 +2046,6 @@ function dim_tag_branch
>  
>       tag=$(tag_name "$branch")
>       tag_branch $tag $branch
> -     git_push $remote $tag
>  }
>  
>  function dim_tag_next
> @@ -2084,7 +2084,6 @@ function dim_pull_request
>               tag=$(tag_name "$branch")
>               gitk --first-parent "$branch" ^$upstream &
>               tag_branch $tag $branch
> -             git_push $remote $tag
>               prep_pull_mail $req_file $tag
>  
>               repo=$(branch_to_repo $branch)
> -- 
> 2.23.0
> 
> _______________________________________________
> dim-tools mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dim-tools

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dim-tools mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dim-tools

Reply via email to