From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:50969)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <weil@mail.berlios.de>) id 1QPIDJ-00006J-5t
	for qemu-devel@nongnu.org; Wed, 25 May 2011 13:49:46 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <weil@mail.berlios.de>) id 1QPIDI-0002wh-58
	for qemu-devel@nongnu.org; Wed, 25 May 2011 13:49:45 -0400
Message-ID: <4DDD4132.1080708@mail.berlios.de>
Date: Wed, 25 May 2011 19:49:38 +0200
From: Stefan Weil <weil@mail.berlios.de>
MIME-Version: 1.0
References: <1306344617-1645-1-git-send-email-peter.maydell@linaro.org>
In-Reply-To: <1306344617-1645-1-git-send-email-peter.maydell@linaro.org>
Content-Type: text/plain; charset=ISO-8859-15; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v2] configure: Don't create symlinks
 to	nonexistent targets
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-trivial@nongnu.org, Stefan Hajnoczi <stefanha@gmail.com>, qemu-devel@nongnu.org, patches@linaro.org

Am 25.05.2011 19:30, schrieb Peter Maydell:
> 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>
> ---
> v2: Remove the debug printing. Oops.
>
> configure | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/configure b/configure
> index a318d37..a648f3d 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" -a ! -e "$f" ]; then
> + symlink "$source_path/$f" "$f"
> + fi
> done
>
> # temporary config to build submodules


Hi,

very good, those bad links which we get today are really annoying,
so for most developers this is an improvement.

Let me add some additional thoughts nevertheless.

* What happens if a git submodule is checked out after configure
   was done? Then the link is missing (until configure is called again).
   Maybe the links should be created from Makefile* instead from
   configure.

* On some restricted hosts which don't create symbolic links (w32),
   the copies which are created instead of links are not updated
   when the original file changes. This is not directly related to
   your change, but could be fixed, too, when the link creation is
   moved to Makefile*.

Cheers,
Stefan W.