[OE-core] [PATCH 1/1] gdb: 7.11 backport of 7.12 avx/mxp support

[Mark Hatle]

On 5/19/16 3:17 PM, bavery wrote:
> Signed-off-by: bavery <brian.avery at intel.com>
> ---
>  meta/recipes-devtools/gdb/gdb-common.inc         |    2 +-
>  meta/recipes-devtools/gdb/gdb/0011-avx_mpx.patch | 2503 ++++++++++++++++++++++
>  2 files changed, 2504 insertions(+), 1 deletion(-)
>  create mode 100644 meta/recipes-devtools/gdb/gdb/0011-avx_mpx.patch
> 
> diff --git a/meta/recipes-devtools/gdb/gdb-common.inc b/meta/recipes-devtools/gdb/gdb-common.inc
> index 1d1cf8f..3e00a2e 100644
> --- a/meta/recipes-devtools/gdb/gdb-common.inc
> +++ b/meta/recipes-devtools/gdb/gdb-common.inc
> @@ -36,6 +36,7 @@ SRC_URI = "git://sourceware.org/git/binutils-gdb.git;branch=gdb-7.11-branch \

...

> diff --git a/meta/recipes-devtools/gdb/gdb/0011-avx_mpx.patch b/meta/recipes-devtools/gdb/gdb/0011-avx_mpx.patch
> new file mode 100644
> index 0000000..f51329d
> --- /dev/null
> +++ b/meta/recipes-devtools/gdb/gdb/0011-avx_mpx.patch

The avx_mpx patch is missing any type of header and sign-off-by.  You need
something to indicate this is a backport, as well as what the purpose of the
patch is.

The patch changelog from 0/1 would be adequate in my opinion as the 'why'.  You
just need an:

Upstream-Status: Backport

...and a signed-off-by line...

--Mark