Am 28.05.2013 10:54, schrieb Paolo Bonzini: > Il 28/05/2013 01:49, Alexey Kardashevskiy ha scritto: >> The symlink to "asm" platform linux headers is made in the build tree by >> the configure script but gcc is not told to look for them there. >> >> The patch fixes this. >> >> Signed-off-by: Alexey Kardashevskiy <a...@ozlabs.ru> >> --- >> >> The previous patch did not remove the old line adding includes. Early >> morning, >> lack of focus :) >> >> --- >> configure | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/configure b/configure >> index 5ae7e4a..eca2a20 100755 >> --- a/configure >> +++ b/configure >> @@ -547,7 +547,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" >> + QEMU_INCLUDES="-I\$(SRC_PATH)/linux-headers -I$(pwd)/linux-headers >> $QEMU_INCLUDES" >> ;; >> esac >> >> > > Reviewed-by: Paolo Bonzini <pbonz...@redhat.com>
Hm, I see no other case of adding $(pwd) to QEMU_INCLUDES - shouldn't this just be -Ilinux-headers? And might it make sense to give the build directory precedence over the source directory? Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
