<div dir="ltr">Nice, <pre>Reviewed-by: Elie Tournier <<a href="mailto:tournier.elie@gmail.com">tournier.elie@gmail.com</a>></pre> <div class="gmail_extra"> <div class="gmail_quote">2016-11-21 18:56 GMT+01:00 Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>>: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>> Mention the script - why/how to use alongside a useful trick to make it work interactively (thanks Rob!). Signed-off-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>> --- 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 &<a href="mailto:lt%3Bjhacker@foo.com">lt;jhacker@foo.com</a>> Acked-by: Joe Hacker &<a href="mailto:lt%3Bjhacker@foo.com">lt;jhacker@foo.com</a>> </pre> +<li>In order for your patch to reach the prospective reviewer easier/faster, +use the script scripts/<a href="http://get_reviewer.pl" rel="noreferrer" target="_blank">get_reviewer.pl</a> to get a list of individuals and include +them in the CC list. + +Please use common sense and do not blindly add everyone. + +<pre> + $ scripts/<a href="http://get_reviewer.pl" rel="noreferrer" target="_blank">get_reviewer.pl</a> --help # to get the the help screen + $ scripts/<a href="http://get_reviewer.pl" rel="noreferrer" target="_blank">get_reviewer.pl</a> -f src/egl/drivers/dri2/platform_android.c + Rob Herring <<a href="mailto:robh@kernel.org">robh@kernel.org</a>> (reviewer:ANDROID EGL SUPPORT,added_lines:188/700=27%,removed_lines:58/283=20%) + Tomasz Figa <<a href="mailto:tfiga@chromium.org">tfiga@chromium.org</a>> (reviewer:ANDROID EGL SUPPORT,authored:12/41=29%,added_lines:308/700=44%,removed_lines:115/283=41%) + Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> (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 <a href="http://get_reviewer.pl" rel="noreferrer" target="_blank">get_reviewer.pl</a> script interactively. Thus you +can avoid adding the world to the CC list. +<pre> + git config sendemail.cccmd "./scripts/<a href="http://get_reviewer.pl" rel="noreferrer" target="_blank">get_reviewer.pl</a> -i" +</pre> </ul> -- 2.10.2 _______________________________________________ mesa-dev mailing list <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> </blockquote></div> </div></div>