script: drop get_reviewer.pl

This script doesn't make sense anymore in the age of GitLab.

Signed-off-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
This commit is contained in:
Eric Engestrom
2019-09-30 22:29:55 +01:00
parent b91ae0379b
commit 721b880e4c
3 changed files with 2 additions and 2349 deletions

View File

@@ -1,30 +1,11 @@
Overview: Overview:
This file is similar in syntax (or more precisly a subset) of what is This file is similar in syntax (or more precisly a subset) of what is
used by the MAINTAINERS file in the linux kernel. Some fields do not used by the MAINTAINERS file in the linux kernel.
apply, for example, in all cases, send patches to:
mesa-dev@lists.freedesktop.org
and in all cases the patchwork instance is:
https://patchwork.freedesktop.org/project/mesa/
The purpose is not exactly the same the MAINTAINERS file in the linux The purpose is not exactly the same the MAINTAINERS file in the linux
kernel, as there are not official/formal maintainers of different kernel, as there are not official/formal maintainers of different
subsystems in mesa, but is meant to give an idea of who to CC for subsystems in mesa, but is meant to give an idea of who to CC for
various patches for review, and to allow the use of various patches for review.
scripts/get_reviewer.pl as git --cc-cmd.
Usage:
When sending patches:
git send-email --cc-cmd ./scripts/get_reviewer.pl ...
Or to configure as default:
git config sendemail.cccmd ./scripts/get_reviewer.pl
Descriptions of section entries: Descriptions of section entries:
@@ -36,14 +17,6 @@ Descriptions of section entries:
F: drivers/net/* all files in drivers/net, but not below F: drivers/net/* all files in drivers/net, but not below
F: */net/* all files in "any top level directory"/net F: */net/* all files in "any top level directory"/net
One pattern per line. Multiple F: lines acceptable. One pattern per line. Multiple F: lines acceptable.
N: Files and directories with regex patterns.
N: [^a-z]tegra all files whose path contains the word tegra
One pattern per line. Multiple N: lines acceptable.
scripts/get_maintainer.pl has different behavior for files that
match F: pattern and matches of N: patterns. By default,
get_maintainer will not look at git log history when an F: pattern
match occurs. When an N: match occurs, git log history is used
to also notify the people that have git commit signatures.
Maintainers List (try to look for most precise areas first) Maintainers List (try to look for most precise areas first)

View File

@@ -122,19 +122,6 @@ that should be documented with:
<li>If sending later revision of a patch, add all the tags - ack, r-b, <li>If sending later revision of a patch, add all the tags - ack, r-b,
Cc: mesa-stable and/or other. This provides reviewers with quick feedback if the Cc: mesa-stable and/or other. This provides reviewers with quick feedback if the
patch has already been reviewed. patch has already been reviewed.
<li>In order for your patch to reach the prospective reviewer easier/faster,
use the script scripts/get_reviewer.pl to get a list of individuals and include
them in the CC list.
<p>
Please use common sense and do <strong>not</strong> blindly add everyone.
</p>
<pre>
$ scripts/get_reviewer.pl --help # to get the help screen
$ scripts/get_reviewer.pl -f src/egl/drivers/dri2/platform_android.c
Rob Herring &lt;robh@kernel.org&gt; (reviewer:ANDROID EGL SUPPORT,added_lines:188/700=27%,removed_lines:58/283=20%)
Tomasz Figa &lt;tfiga@chromium.org&gt; (reviewer:ANDROID EGL SUPPORT,authored:12/41=29%,added_lines:308/700=44%,removed_lines:115/283=41%)
Emil Velikov &lt;emil.l.velikov@gmail.com&gt; (authored:13/41=32%,removed_lines:76/283=27%)
</pre>
</ul> </ul>
@@ -493,11 +480,6 @@ commits before sending.
git send-email --subject-prefix="PATCH v4" HEAD~8 git send-email --subject-prefix="PATCH v4" HEAD~8
git send-email -v4 @~8 # shorter version, inherited from git format-patch git send-email -v4 @~8 # shorter version, inherited from git format-patch
</pre> </pre>
<li> Configure git to use the get_reviewer.pl script interactively. Thus you
can avoid adding the world to the CC list.
<pre>
git config sendemail.cccmd "./scripts/get_reviewer.pl -i"
</pre>
</ul> </ul>

File diff suppressed because it is too large Load Diff