[Mesa-dev] [PATCH] docs/submitting patches: mention get_reviewers.pl
Emil Velikov
emil.l.velikov at gmail.com
Mon Nov 21 17:56:08 UTC 2016
From: Emil Velikov <emil.velikov at collabora.com>
Mention the script - why/how to use alongside a useful trick to make it
work interactively (thanks Rob!).
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
---
Based on top of the "git tips" from earlier.
---
docs/submittingpatches.html | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/docs/submittingpatches.html b/docs/submittingpatches.html
index 7ada508..2f4be5d 100644
--- a/docs/submittingpatches.html
+++ b/docs/submittingpatches.html
@@ -104,6 +104,19 @@ that should be documented with:
Reviewed-by: Joe Hacker <jhacker at foo.com>
Acked-by: Joe Hacker <jhacker at foo.com>
</pre>
+<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.
+<br>
+Please use common sense and do <strong>not</strong> blindly add everyone.
+<br>
+<pre>
+ $ scripts/get_reviewer.pl --help # to get the the help screen
+ $ scripts/get_reviewer.pl -f src/egl/drivers/dri2/platform_android.c
+ Rob Herring <robh at kernel.org> (reviewer:ANDROID EGL SUPPORT,added_lines:188/700=27%,removed_lines:58/283=20%)
+ Tomasz Figa <tfiga at chromium.org> (reviewer:ANDROID EGL SUPPORT,authored:12/41=29%,added_lines:308/700=44%,removed_lines:115/283=41%)
+ Emil Velikov <emil.l.velikov at gmail.com> (authored:13/41=32%,removed_lines:76/283=27%)
+</pre>
</ul>
@@ -335,6 +348,11 @@ commits before sending.
git send-email --subject-prefix="PATCH v4" HEAD~8
git send-email -v4 HEAD~8 # shorter version, 'inherited' from git format-patch
</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>
--
2.10.2
More information about the mesa-dev
mailing list