Patchwork configure: adapt vde test to -Werror

login
register
mail settings
Submitter Laszlo Ersek
Date July 15, 2012, 7:38 p.m.
Message ID <1342381105-20395-1-git-send-email-lersek@redhat.com>
Download mbox | patch
Permalink /patch/171088/
State New
Headers show

Comments

Laszlo Ersek - July 15, 2012, 7:38 p.m.
... after commit 417c9d72 ("configure: add -Werror to QEMU_CFLAGS early")

Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
 configure |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Stefan Hajnoczi - July 16, 2012, 11:10 a.m.
On Sun, Jul 15, 2012 at 09:38:25PM +0200, Laszlo Ersek wrote:
> ... after commit 417c9d72 ("configure: add -Werror to QEMU_CFLAGS early")
> 
> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
> ---
>  configure |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan
Stefan Hajnoczi - July 21, 2012, 9:54 a.m.
On Mon, Jul 16, 2012 at 12:10 PM, Stefan Hajnoczi <stefanha@gmail.com> wrote:
> On Sun, Jul 15, 2012 at 09:38:25PM +0200, Laszlo Ersek wrote:
>> ... after commit 417c9d72 ("configure: add -Werror to QEMU_CFLAGS early")
>>
>> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
>> ---
>>  configure |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> Thanks, applied to the trivial patches tree:
> https://github.com/stefanha/qemu/commits/trivial-patches

Peter Maydell's 11 patch series addresses all the other ./configure
-Werror issues too.  You were first but I'm dropping this so
qemu.git/master can solve all issues at once with Peter's series.

Stefan

Patch

diff --git a/configure b/configure
index 0a3896e..8a1b902 100755
--- a/configure
+++ b/configure
@@ -1819,7 +1819,7 @@  if test "$vde" != "no" ; then
 int main(void)
 {
     struct vde_open_args a = {0, 0, 0};
-    vde_open("", "", &a);
+    vde_open((char *)"", (char *)"", &a);
     return 0;
 }
 EOF