* [Qemu-devel] [PATCH] configure: check for -Wendif-labels support
@ 2011-05-17 21:08 Mike Frysinger
2011-05-18 7:45 ` Stefan Hajnoczi
2011-06-03 21:50 ` Aurelien Jarno
0 siblings, 2 replies; 3+ messages in thread
From: Mike Frysinger @ 2011-05-17 21:08 UTC (permalink / raw)
To: qemu-devel
Older gcc compilers do not support -Wendif-labels, so move it from the
hardcoded list to the dynamically detected list.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
configure | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index 019bc8d..8b775a4 100755
--- a/configure
+++ b/configure
@@ -229,7 +229,7 @@ sdl_config="${cross_prefix}${SDL_CONFIG-sdl-config}"
# default flags for all hosts
QEMU_CFLAGS="-fno-strict-aliasing $QEMU_CFLAGS"
CFLAGS="-g $CFLAGS"
-QEMU_CFLAGS="-Wall -Wundef -Wendif-labels -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
+QEMU_CFLAGS="-Wall -Wundef -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
QEMU_CFLAGS="-Wstrict-prototypes -Wredundant-decls $QEMU_CFLAGS"
QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS"
QEMU_CFLAGS="-D_FORTIFY_SOURCE=2 $QEMU_CFLAGS"
@@ -1035,7 +1035,7 @@ fi
gcc_flags="-Wold-style-declaration -Wold-style-definition -Wtype-limits"
gcc_flags="-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers $gcc_flags"
gcc_flags="-Wmissing-include-dirs -Wempty-body -Wnested-externs $gcc_flags"
-gcc_flags="-fstack-protector-all $gcc_flags"
+gcc_flags="-fstack-protector-all -Wendif-labels $gcc_flags"
cat > $TMPC << EOF
int main(void) { return 0; }
EOF
--
1.7.5.rc3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] configure: check for -Wendif-labels support
2011-05-17 21:08 [Qemu-devel] [PATCH] configure: check for -Wendif-labels support Mike Frysinger
@ 2011-05-18 7:45 ` Stefan Hajnoczi
2011-06-03 21:50 ` Aurelien Jarno
1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2011-05-18 7:45 UTC (permalink / raw)
To: Mike Frysinger; +Cc: qemu-devel
On Tue, May 17, 2011 at 10:08 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> Older gcc compilers do not support -Wendif-labels, so move it from the
> hardcoded list to the dynamically detected list.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> configure | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] configure: check for -Wendif-labels support
2011-05-17 21:08 [Qemu-devel] [PATCH] configure: check for -Wendif-labels support Mike Frysinger
2011-05-18 7:45 ` Stefan Hajnoczi
@ 2011-06-03 21:50 ` Aurelien Jarno
1 sibling, 0 replies; 3+ messages in thread
From: Aurelien Jarno @ 2011-06-03 21:50 UTC (permalink / raw)
To: Mike Frysinger; +Cc: qemu-devel
On Tue, May 17, 2011 at 05:08:43PM -0400, Mike Frysinger wrote:
> Older gcc compilers do not support -Wendif-labels, so move it from the
> hardcoded list to the dynamically detected list.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> configure | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
Thanks, applied.
> diff --git a/configure b/configure
> index 019bc8d..8b775a4 100755
> --- a/configure
> +++ b/configure
> @@ -229,7 +229,7 @@ sdl_config="${cross_prefix}${SDL_CONFIG-sdl-config}"
> # default flags for all hosts
> QEMU_CFLAGS="-fno-strict-aliasing $QEMU_CFLAGS"
> CFLAGS="-g $CFLAGS"
> -QEMU_CFLAGS="-Wall -Wundef -Wendif-labels -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
> +QEMU_CFLAGS="-Wall -Wundef -Wwrite-strings -Wmissing-prototypes $QEMU_CFLAGS"
> QEMU_CFLAGS="-Wstrict-prototypes -Wredundant-decls $QEMU_CFLAGS"
> QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS"
> QEMU_CFLAGS="-D_FORTIFY_SOURCE=2 $QEMU_CFLAGS"
> @@ -1035,7 +1035,7 @@ fi
> gcc_flags="-Wold-style-declaration -Wold-style-definition -Wtype-limits"
> gcc_flags="-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers $gcc_flags"
> gcc_flags="-Wmissing-include-dirs -Wempty-body -Wnested-externs $gcc_flags"
> -gcc_flags="-fstack-protector-all $gcc_flags"
> +gcc_flags="-fstack-protector-all -Wendif-labels $gcc_flags"
> cat > $TMPC << EOF
> int main(void) { return 0; }
> EOF
> --
> 1.7.5.rc3
>
>
>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
aurelien@aurel32.net http://www.aurel32.net
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-06-03 21:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-17 21:08 [Qemu-devel] [PATCH] configure: check for -Wendif-labels support Mike Frysinger
2011-05-18 7:45 ` Stefan Hajnoczi
2011-06-03 21:50 ` Aurelien Jarno
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.