[Intel-gfx] [maintainer-tools PATCH 5/5] dim: update pull and test request recipients

[Sean Paul]

On Fri, Sep 8, 2017 at 9:30 AM, Jani Nikula <jani.nikula at intel.com> wrote:
> Make it easier to add/remove people as needed. Add Joonas, Rodrigo and
> Sean as new recipients for pull requests. Replace Daniel with Joonas and
> Rodrigo as test request recipients.
>
> Perhaps we'll want to make the list branch specific in the future, but
> get the low hanging fruit first.
>
> Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Cc: Sean Paul <seanpaul at chromium.org>

Hi Jani,
For the entire series:

Reviewed-by: Sean Paul <seanpaul at chromium.org> (although this means
I'll have to add default signatures now :/)



> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
>  dim | 41 +++++++++++++++++++++++------------------
>  1 file changed, 23 insertions(+), 18 deletions(-)
>
> diff --git a/dim b/dim
> index 96baffde5524..b177b9b28242 100755
> --- a/dim
> +++ b/dim
> @@ -83,13 +83,27 @@ drm_tip_ssh=ssh://git.freedesktop.org/git/drm-tip
>  drm_upstream_git=git://people.freedesktop.org/~airlied/linux
>  linux_upstream_git=git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>
> -# email aliases
> -addr_drm_maintainer="Dave Airlie <airlied at gmail.com>"
> -addr_intel_gfx_maintainer1="Daniel Vetter <daniel.vetter at ffwll.ch>"
> -addr_intel_gfx_maintainer2="Jani Nikula <jani.nikula at linux.intel.com>"
> -addr_intel_gfx="intel-gfx at lists.freedesktop.org"
> -addr_dri_devel="dri-devel at lists.freedesktop.org"
> -addr_intel_qa="\"Jari Tahvanainen\" <jari.tahvanainen at intel.com>"
> +# Recipients for all dim based pull requests.
> +# Add To: lines to the end, Cc: lines in the beginning with -c.
> +pull_request_recipients=(
> +       -c "Daniel Vetter <daniel.vetter at ffwll.ch>"
> +       -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 "Sean Paul <seanpaul at chromium.org>"
> +       -c "dri-devel at lists.freedesktop.org"
> +       -c "intel-gfx at lists.freedesktop.org"
> +       "Dave Airlie <airlied at gmail.com>"
> +)
> +
> +# Recipients for drm-intel-testing updates.
> +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
>  integration_config=nightly.conf
> @@ -1557,11 +1571,7 @@ function dim_update_next_continue
>         prep_pull_mail_signature >> $req_file
>
>         $DRY $DIM_MUA -s "Updated drm-intel-testing" \
> -            -i $req_file \
> -            -c "$addr_intel_gfx" \
> -            -c "$addr_intel_gfx_maintainer1" \
> -            -c "$addr_intel_gfx_maintainer2" \
> -            "$addr_intel_qa"
> +            -i $req_file "${test_request_recipients[@]}"
>  }
>
>  function dim_tag_next
> @@ -1652,12 +1662,7 @@ function dim_pull_request
>
>         git request-pull $upstream $git_url $tag >> $req_file
>         $DRY $DIM_MUA -s "[PULL] $branch" \
> -               -i $req_file \
> -               -c "$addr_intel_gfx" \
> -               -c "$addr_dri_devel" \
> -               -c "$addr_intel_gfx_maintainer1" \
> -               -c "$addr_intel_gfx_maintainer2" \
> -               "$addr_drm_maintainer"
> +            -i $req_file "${pull_request_recipients[@]}"
>  }
>
>  function dim_pull_request_next
> --
> 2.11.0
>