[OE-core] [PATCH] libsdl2: disable check gcc -Wdeclaration-after-statement for ppc

[Richard Purdie]

On Mon, 2017-04-10 at 11:14 +0800, kai.kang at windriver.com wrote:
> From: Kai Kang <kai.kang at windriver.com>
> 
> Disable check gcc option -Wdeclaration-after-statement for powerpc
> and
> powerpc64 which causes compile failures:
> 
> > 
> >   CC     build/SDL_blit_N.lo
> > .../tmp/work/ppc7400-wrs-linux/libsdl2/2.0.5-r0/SDL2-
> > 2.0.5/src/video/SDL_blit_N.c: In function 'calc_swizzle32':
> > .../tmp/work/ppc7400-wrs-linux/libsdl2/2.0.5-r0/SDL2-
> > 2.0.5/src/video/SDL_blit_N.c:127:5: error: ISO C90 forbids
> > mixed declarations and code [-Werror=declaration-after-statement]
> >      const vector unsigned char plus = VECUINT8_LITERAL(0x00, 0x00,
> > 0x00, 0x00,
> >      ^~~~~
> Signed-off-by: Kai Kang <kai.kang at windriver.com>
> ---
>  ...able-check-gcc-Wdeclaration-after-statement.patch | 20
> ++++++++++++++++++++
>  meta/recipes-graphics/libsdl2/libsdl2_2.0.5.bb       |  2 ++
>  2 files changed, 22 insertions(+)
>  create mode 100644 meta/recipes-graphics/libsdl2/libsdl2/disable-
> check-gcc-Wdeclaration-after-statement.patch
> 
> diff --git a/meta/recipes-graphics/libsdl2/libsdl2/disable-check-gcc-
> Wdeclaration-after-statement.patch b/meta/recipes-
> graphics/libsdl2/libsdl2/disable-check-gcc-Wdeclaration-after-
> statement.patch
> new file mode 100644
> index 0000000..ad48af7
> --- /dev/null
> +++ b/meta/recipes-graphics/libsdl2/libsdl2/disable-check-gcc-
> Wdeclaration-after-statement.patch
> @@ -0,0 +1,20 @@
> +Upstream-Status: Inappropriate [disable feature]

Given upstream explicitly enable this warning and the issue looks to be
in their code, surely upstream would want to fix this.

Also, would it not be easier to patch the code and fix this than
disabling the warning?

Cheers,

Richard


> +Disable check gcc option -Wdeclaration-after-statement for powerpc
> and
> +powerpc64 which causes compile failures.
> +
> +Signed-off-by: Kai Kang <kai.kang at windriver.com>
> +---
> +diff --git a/configure.in b/configure.in
> +index e2ddc9c..73b10f5 100644
> +--- a/configure.in
> ++++ b/configure.in
> +@@ -2950,7 +2950,7 @@ case "$host" in
> +             *-*-minix*)         ARCH=minix ;;
> +         esac
> +         CheckVisibilityHidden
> +-        CheckDeclarationAfterStatement
> ++#        CheckDeclarationAfterStatement
> +         CheckDummyVideo
> +         CheckDiskAudio
> +         CheckDummyAudio
> diff --git a/meta/recipes-graphics/libsdl2/libsdl2_2.0.5.bb
> b/meta/recipes-graphics/libsdl2/libsdl2_2.0.5.bb
> index d747cf1..33963c1 100644
> --- a/meta/recipes-graphics/libsdl2/libsdl2_2.0.5.bb
> +++ b/meta/recipes-graphics/libsdl2/libsdl2_2.0.5.bb
> @@ -20,6 +20,8 @@ SRC_URI = " \
>      file://0001-prepend-our-sysroot-path-so-that-make-finds-our-
> wayl.patch \
>      file://0002-Avoid-finding-build-host-s-wayland-scanner.patch \
>  "
> +SRC_URI_append_powerpc = " file://disable-check-gcc-Wdeclaration-
> after-statement.patch"
> +SRC_URI_append_powerpc64 = " file://disable-check-gcc-Wdeclaration-
> after-statement.patch"
>  
>  S = "${WORKDIR}/SDL2-${PV}"
>  
> -- 
> 2.10.1
>