[Qemu-devel] [PATCH] configure: avoid compiler warning in pipe2 detection

[Qemu-devel] [PATCH] configure: avoid compiler warning in pipe2 detection

[Bruce Rogers]

When building qemu-kvm for openSUSE:Factory, I am getting a
warning in the pipe2 detection performed by configure, which
prevents using --enable-werror.

Change detection code to use return value of pipe2.

Signed-off-by: Bruce Rogers <brogers@suse.com>
---
 configure |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index 5808764..2e1f7f8 100755
--- a/configure
+++ b/configure
@@ -2399,8 +2399,7 @@ cat > $TMPC << EOF
 int main(void)
 {
     int pipefd[2];
-    pipe2(pipefd, O_CLOEXEC);
-    return 0;
+    return pipe2(pipefd, O_CLOEXEC);
 }
 EOF
 if compile_prog "" "" ; then
-- 
1.7.7

Re: [Qemu-devel] [PATCH] configure: avoid compiler warning in pipe2 detection

[Peter Maydell]

On 20 August 2012 19:45, Bruce Rogers <brogers@suse.com> wrote:
> When building qemu-kvm for openSUSE:Factory, I am getting a
> warning in the pipe2 detection performed by configure, which
> prevents using --enable-werror.
>
> Change detection code to use return value of pipe2.
>
> Signed-off-by: Bruce Rogers <brogers@suse.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

-- PMM

Re: [Qemu-devel] [PATCH] configure: avoid compiler warning in pipe2 detection

[Peter Maydell]

Ping! Just ran into this bug today and noticed this
patch never got applied.
Patch still applies OK, patchwork url is:
http://patchwork.ozlabs.org/patch/178920/

thanks
-- PMM

On 20 August 2012 19:45, Bruce Rogers <brogers@suse.com> wrote:
> When building qemu-kvm for openSUSE:Factory, I am getting a
> warning in the pipe2 detection performed by configure, which
> prevents using --enable-werror.
>
> Change detection code to use return value of pipe2.
>
> Signed-off-by: Bruce Rogers <brogers@suse.com>
> ---
>  configure |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 5808764..2e1f7f8 100755
> --- a/configure
> +++ b/configure
> @@ -2399,8 +2399,7 @@ cat > $TMPC << EOF
>  int main(void)
>  {
>      int pipefd[2];
> -    pipe2(pipefd, O_CLOEXEC);
> -    return 0;
> +    return pipe2(pipefd, O_CLOEXEC);
>  }
>  EOF
>  if compile_prog "" "" ; then
> --
> 1.7.7
>
>

Re: [Qemu-devel] [PATCH] configure: avoid compiler warning in pipe2 detection

[Andreas Färber]

Am 24.10.2012 18:48, schrieb Peter Maydell:
> Ping! Just ran into this bug today and noticed this
> patch never got applied.
> Patch still applies OK, patchwork url is:
> http://patchwork.ozlabs.org/patch/178920/

...and since it was noticed while packaging v1.2, it should go into
stable as well (cc'ed).

Andreas

> On 20 August 2012 19:45, Bruce Rogers <brogers@suse.com> wrote:
>> When building qemu-kvm for openSUSE:Factory, I am getting a
>> warning in the pipe2 detection performed by configure, which
>> prevents using --enable-werror.
>>
>> Change detection code to use return value of pipe2.
>>
>> Signed-off-by: Bruce Rogers <brogers@suse.com>
>> ---
>>  configure |    3 +--
>>  1 files changed, 1 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 5808764..2e1f7f8 100755
>> --- a/configure
>> +++ b/configure
>> @@ -2399,8 +2399,7 @@ cat > $TMPC << EOF
>>  int main(void)
>>  {
>>      int pipefd[2];
>> -    pipe2(pipefd, O_CLOEXEC);
>> -    return 0;
>> +    return pipe2(pipefd, O_CLOEXEC);
>>  }
>>  EOF
>>  if compile_prog "" "" ; then
>> --
>> 1.7.7

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-devel] [PATCH] configure: avoid compiler warning in pipe2 detection

[Blue Swirl]

Thanks, applied.

On Mon, Aug 20, 2012 at 6:45 PM, Bruce Rogers <brogers@suse.com> wrote:
> When building qemu-kvm for openSUSE:Factory, I am getting a
> warning in the pipe2 detection performed by configure, which
> prevents using --enable-werror.
>
> Change detection code to use return value of pipe2.
>
> Signed-off-by: Bruce Rogers <brogers@suse.com>
> ---
>  configure |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 5808764..2e1f7f8 100755
> --- a/configure
> +++ b/configure
> @@ -2399,8 +2399,7 @@ cat > $TMPC << EOF
>  int main(void)
>  {
>      int pipefd[2];
> -    pipe2(pipefd, O_CLOEXEC);
> -    return 0;
> +    return pipe2(pipefd, O_CLOEXEC);
>  }
>  EOF
>  if compile_prog "" "" ; then
> --
> 1.7.7
>
>