Re: [Qemu-trivial] [Qemu-devel] [PATCH v3] configure: Don't create symlinks to nonexistent targets

Re: [Qemu-trivial] [Qemu-devel] [PATCH v3] configure: Don't create symlinks to nonexistent targets

[Peter Maydell]

ping?

I didn't cc trivial first time round on the theory that any patch
that makes it to v3 isn't trivial, but since nobody's commented
on this version I guess it's OK...

-- PMM

On 3 June 2011 17:10, Peter Maydell <peter.maydell@linaro.org> wrote:
> When we create the symlinks to source tree files, don't create them
> if the file is not actually present in the source tree; this will
> happen if the file is in a git submodule that wasn't checked out.
>
> This also avoids the odd effect where an in-source-tree configure
> will end up creating the missing file as a symlink to itself.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> v1->v2: remove debug printing
> v2->v3: don't use test constructs marked by POSIX as obsolete,
> as pointed out by Paolo Bonzini
>
>  configure |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/configure b/configure
> index a318d37..7f8ad24 100755
> --- a/configure
> +++ b/configure
> @@ -3564,7 +3564,9 @@ for bios_file in $source_path/pc-bios/*.bin $source_path/pc-bios/*.rom $source_p
>  done
>  mkdir -p $DIRS
>  for f in $FILES ; do
> -    test -e $f || symlink $source_path/$f $f
> +    if [ -e "$source_path/$f" ] && ! [ -e "$f" ]; then
> +        symlink "$source_path/$f" "$f"
> +    fi
>  done
>
>  # temporary config to build submodules
> --
> 1.7.1