qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Add linux-headers to QEMU_INCLUDES
@ 2013-04-10  8:29 Jan Kiszka
  2013-04-10  9:44 ` Paolo Bonzini
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Kiszka @ 2013-04-10  8:29 UTC (permalink / raw)
  To: Anthony Liguori; +Cc: Paolo Bonzini, qemu-devel

From: Jan Kiszka <jan.kiszka@siemens.com>

virtio/dataplane/vring.c requires the Linux headers and is built for all
targets. So we need to add the corresponding include to QEMU_INCLUDES to
avoid that outdated distribution headers are used.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 configure |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/configure b/configure
index 1ed939a..ee07f07 100755
--- a/configure
+++ b/configure
@@ -544,6 +544,7 @@ Haiku)
   if [ "$cpu" = "i386" -o "$cpu" = "x86_64" ] ; then
     audio_possible_drivers="$audio_possible_drivers fmod"
   fi
+  QEMU_INCLUDES="-I\$(SRC_PATH)/linux-headers $QEMU_INCLUDES"
 ;;
 esac
 
@@ -4230,10 +4231,6 @@ else
 fi
 includes="-I\$(SRC_PATH)/tcg $includes"
 
-if test "$linux" = "yes" ; then
-  includes="-I\$(SRC_PATH)/linux-headers $includes"
-fi
-
 for i in $ARCH $TARGET_BASE_ARCH ; do
   case "$i" in
   alpha)
-- 
1.7.3.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] Add linux-headers to QEMU_INCLUDES
  2013-04-10  8:29 [Qemu-devel] [PATCH] Add linux-headers to QEMU_INCLUDES Jan Kiszka
@ 2013-04-10  9:44 ` Paolo Bonzini
  0 siblings, 0 replies; 2+ messages in thread
From: Paolo Bonzini @ 2013-04-10  9:44 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: Anthony Liguori, qemu-devel

Il 10/04/2013 10:29, Jan Kiszka ha scritto:
> From: Jan Kiszka <jan.kiszka@siemens.com>
> 
> virtio/dataplane/vring.c requires the Linux headers and is built for all
> targets. So we need to add the corresponding include to QEMU_INCLUDES to
> avoid that outdated distribution headers are used.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>  configure |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/configure b/configure
> index 1ed939a..ee07f07 100755
> --- a/configure
> +++ b/configure
> @@ -544,6 +544,7 @@ Haiku)
>    if [ "$cpu" = "i386" -o "$cpu" = "x86_64" ] ; then
>      audio_possible_drivers="$audio_possible_drivers fmod"
>    fi
> +  QEMU_INCLUDES="-I\$(SRC_PATH)/linux-headers $QEMU_INCLUDES"
>  ;;
>  esac
>  
> @@ -4230,10 +4231,6 @@ else
>  fi
>  includes="-I\$(SRC_PATH)/tcg $includes"
>  
> -if test "$linux" = "yes" ; then
> -  includes="-I\$(SRC_PATH)/linux-headers $includes"
> -fi
> -
>  for i in $ARCH $TARGET_BASE_ARCH ; do
>    case "$i" in
>    alpha)
> 

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-04-10  9:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-10  8:29 [Qemu-devel] [PATCH] Add linux-headers to QEMU_INCLUDES Jan Kiszka
2013-04-10  9:44 ` Paolo Bonzini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).