Re: [Qemu-devel] [PATCH] qemu: fix out of tree cross compile

[Peter Maydell <peter.maydell@linaro.org>]

On 27 May 2013 07:13, Alexey Kardashevskiy <aik@ozlabs.ru> wrote:
> QEMU uses headers from source directory, however the "asm" symlink was
> created in the build directory what broke KVM stuff compilation.
>
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> ---
>  configure |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index 5ae7e4a..9c3c5e5 100755
> --- a/configure
> +++ b/configure
> @@ -4135,7 +4135,7 @@ if test "$linux" = "yes" ; then
>    esac
>      # For non-KVM architectures we will not have asm headers
>      if [ -e "$source_path/linux-headers/asm-$linux_arch" ]; then
> -      symlink "$source_path/linux-headers/asm-$linux_arch" linux-headers/asm
> +      symlink "$source_path/linux-headers/asm-$linux_arch" $source_path/linux-headers/asm

This is definitely wrong -- the asm symlink must be created
in the build tree, because you could have two build trees
which are building for different architectures and need
different symlink targets. (More generally, an out of tree
build should never create files in the source tree, because
the point of out of tree builds is to leave the source tree
in a pristine state.)

What is the actual bug here? QEMU builds fine for me, because
the build directory is also in the include path via "-I.".

thanks
-- PMM