[dim PATCH 2/7] dim: stop generating drm-intel-testing requests

Daniel Vetter daniel at ffwll.ch
Thu Dec 6 08:36:45 UTC 2018


On Wed, Dec 05, 2018 at 05:02:54PM +0200, Jani Nikula wrote:
> Since the drm-intel-testing branch was introduced, we've completely
> shifted from testing pull requests to pre-merge patch testing and
> continuous testing. The drm-intel-testing branch and testing request has
> outlived its usefulness. Remove it.
> 
> We still leave the workflow for tagging and making pull requests as they
> are for now.
> 
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
>  dim     | 52 ++++------------------------------------------------
>  dim.rst |  9 ++-------
>  2 files changed, 6 insertions(+), 55 deletions(-)
> 
> diff --git a/dim b/dim
> index 8187f8894c8f..8e782eecff70 100755
> --- a/dim
> +++ b/dim
> @@ -94,15 +94,6 @@ dim_pull_request_recipients=(
>  	"Dave Airlie <airlied at gmail.com>"
>  )
>  
> -# Recipients for drm-intel-testing updates.
> -dim_test_request_recipients=(
> -	-c "Jani Nikula <jani.nikula at linux.intel.com>"
> -	-c "Joonas Lahtinen <joonas.lahtinen at linux.intel.com>"
> -	-c "Rodrigo Vivi <rodrigo.vivi at intel.com>"
> -	-c "intel-gfx at lists.freedesktop.org"
> -	"Jari Tahvanainen <jari.tahvanainen at intel.com>"
> -)
> -
>  # integration configuration
>  dim_integration_config=nightly.conf
>  
> @@ -1860,23 +1851,9 @@ function dim_update_driver_date
>  
>  function dim_update_next
>  {
> -	local remote
> -
>  	assert_branch drm-intel-next-queued
>  
> -	remote=$(repo_to_remote drm-tip)
> -
>  	git pull --ff-only
> -	git fetch drm-tip
> -
> -	if ! git branch --merged $remote/drm-tip | grep -q drm-intel-fixes ; then
> -		echo "drm-intel-fixes not merged into drm-tip, please update!"
> -		exit 2
> -	fi
> -	if ! git branch --merged $remote/drm-tip | grep -q drm-intel-next-queued ; then
> -		echo "drm-intel-next-queued not merged into drm-tip, please update!"
> -		exit 2
> -	fi

Yeah this was essentially for me to make sure I didn't forget to push out
anything. Kinda doesn't work with group maintainership anymore, can't test
other people's local trees :-)
>  
>  	dim_update_driver_date "drm/i915" "drivers/gpu/drm/i915/i915_drv.h"
>  
> @@ -1890,38 +1867,17 @@ function dim_update_next
>  
>  function dim_update_next_continue
>  {
> -	local remote intel_remote req_file tag tag_testing
> +	local remote tag
>  
>  	assert_branch drm-intel-next-queued
>  
> -	intel_remote=$(repo_to_remote drm-intel)
> -	remote=$(repo_to_remote drm-tip)
> +	remote=$(repo_to_remote drm-intel)
>  
> -	git push $DRY_RUN -f $intel_remote drm-intel-next-queued:drm-intel-next
> +	git push $DRY_RUN -f $remote drm-intel-next-queued:drm-intel-next
>  	tag=$(tag_name "drm-intel-next")
> -	tag_testing="${tag/next/testing}"
>  
>  	tag_branch $tag drm-intel-next
> -	git push $DRY_RUN $intel_remote $tag
> -
> -	echo "Updating drm-intel-testing to latest drm-tip"
> -	git push $DRY_RUN $intel_remote +$remote/drm-tip:drm-intel-testing
> -	$DRY git tag $tag_testing $intel_remote/drm-intel-testing
> -	$DRY git push $intel_remote $tag_testing
> -
> -	req_file=$(mktemp)
> -	cat > $req_file <<-HERE
> -		Hi all,
> -
> -		The following changes tagged $tag_testing:
> -
> -		HERE
> -
> -	prep_pull_mail_overview "$tag" >> $req_file
> -	prep_pull_mail_signature >> $req_file
> -
> -	$DRY $DIM_MUA -s "Updated drm-intel-testing" \
> -	     -i $req_file "${dim_test_request_recipients[@]}"
> +	git push $DRY_RUN $remote $tag

Calling into dim_tag_branch instead to also share the git push?

Either way: Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>
>  }
>  
>  function dim_tag_branch
> diff --git a/dim.rst b/dim.rst
> index 2a0deedb6ba1..7aa90e659983 100644
> --- a/dim.rst
> +++ b/dim.rst
> @@ -362,13 +362,8 @@ should be preferred.
>  
>  update-next
>  -----------
> -Pushes out the latest dinq to drm-intel-next and tags it. Also
> -pushes out the latest drm-tip to drm-intel-testing. For an
> -overview a gitk view of the currently unmerged feature pile is
> -opened.
> -
> -Also checks that the drm-intel-fixes|-next-queued are fully
> -merged into drm-tip to avoid operator error.
> +Pushes out the latest dinq to drm-intel-next and tags it. For an overview a gitk
> +view of the currently unmerged feature pile is opened.
>  
>  The tag will be signed using the key specified by \$DIM_GPG_KEYID, if set.
>  
> -- 
> 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


More information about the dim-tools mailing list