On 24 January 2017 at 00:40, Peter Hutterer <peter.hutte...@who-t.net> wrote:
> From: Adam Jackson <a...@redhat.com>
>
> Per discussion at XDC2015, we want this so we can easily distinguish
> which module a patch is for. There's no way to set this in the
> server-side config, so setting a default at autogen time is about the
> best we can do.
>
> Reviewed-by: Eric Anholt <e...@anholt.net>
> Signed-off-by: Adam Jackson <a...@redhat.com>
> Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
> ---
>  autogen.sh | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/autogen.sh b/autogen.sh
> index 0006de8..cda30cc 100755
> --- a/autogen.sh
> +++ b/autogen.sh
> @@ -9,6 +9,9 @@ cd "$srcdir"
>  autoreconf -v --install || exit 1
>  cd "$ORIGDIR" || exit $?
>
> +git config --local --get format.subjectPrefix ||
> +    git config --local format.subjectPrefix "PATCH bigreqsproto"
Seems off, plus a simple basename `pwd` might not always work.

Mihail Konev has a patch (message id
<20161014000652.GA12465@localhost>) which includes a tarball with all
the patches.
From a quick skim things seem ok - don't see anything wrong and/or
missing. Fwiw I'd go with that one.

The rest of the series is (fwiw)
Reviewed-by: Emil Velikov <emil.veli...@collabora.com>

Thanks !
Emil
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to