On Tue, 2017-04-11 at 10:46 +0200, Juan A. Suarez Romero wrote:
> Both scripts does not use a file with the commits to ignore. So if we
> have handled one of the suggested commits and decided we won't pick it,
> the scripts will continue suggesting them.
> 


Emil, if you're fine with this version, can you grant a R-b?

Thanks!


> v2:
> - Mark the candidates in bin/get-extra-pick-list.sh (Juan A. Suarez)
> - Use bin/.cherry-ignore to store rejected patches (Emil)
> ---
>  bin/get-extra-pick-list.sh | 6 ++++++
>  bin/get-fixes-pick-list.sh | 7 +++++++
>  2 files changed, 13 insertions(+)
> 
> diff --git a/bin/get-extra-pick-list.sh b/bin/get-extra-pick-list.sh
> index 4d39389..dbd5474 100755
> --- a/bin/get-extra-pick-list.sh
> +++ b/bin/get-extra-pick-list.sh
> @@ -30,6 +30,12 @@ do
>               if grep -q ^$candidate already_picked ; then
>                       continue
>               fi
> +             # Or if it isn't in the ignore list.
> +             if [ -f bin/.cherry-ignore ] ; then
> +                     if grep -q ^$candidate bin/.cherry-ignore ; then
> +                             continue
> +                     fi
> +             fi
>               echo Commit $candidate references $sha
>       done
>  done
> diff --git a/bin/get-fixes-pick-list.sh b/bin/get-fixes-pick-list.sh
> index 75242a2..73afd0c 100755
> --- a/bin/get-fixes-pick-list.sh
> +++ b/bin/get-fixes-pick-list.sh
> @@ -24,6 +24,13 @@ git log --reverse --grep="cherry picked from commit" 
> $latest_branchpoint..HEAD |
>  git log --reverse --pretty=%H -i --grep="fixes:" 
> $latest_branchpoint..origin/master |\
>  while read sha
>  do
> +     # Check to see whether the patch is on the ignore list ...
> +     if [ -f bin/.cherry-ignore ] ; then
> +             if grep -q ^$sha bin/.cherry-ignore ; then
> +                     continue
> +             fi
> +     fi
> +
>       # For each one try to extract the tag
>       fixes_count=`git show $sha | grep -i "fixes:" | wc -l`
>       if [ "x$fixes_count" != x1 ] ; then
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to