[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Makefile: Add missing rule for config-devices.
From: |
Scott Tsai |
Subject: |
Re: [Qemu-devel] [PATCH] Makefile: Add missing rule for config-devices.mak |
Date: |
Sun, 22 Nov 2009 22:20:34 +0800 |
On Sun, Nov 22, 2009 at 10:07 PM, Stefan Weil <address@hidden> wrote:
> All files config-devices.mak are copies from files in
> directory default-configs.
>
> Creating these copies during configuration ignores these
> dependencies and may result in unresolved externals
> from incremental builds when files in default-configs
> are modified.
>
> By adding a dependency rule to Makefile and executing
> the copy commands there it is possible to run incremental
> builds even with modified default-configs.
>
> This is especially useful for git bisect.
>
> Signed-off-by: Stefan Weil <address@hidden>
> ---
> Makefile | 3 +++
> configure | 4 ----
> 2 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index d770e2a..fdb47fb 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -41,6 +41,9 @@ SUBDIR_DEVICES_MAK=$(patsubst %, %/config-devices.mak,
> $(TARGET_DIRS))
> config-all-devices.mak: $(SUBDIR_DEVICES_MAK)
> $(call quiet-command,cat $(SUBDIR_DEVICES_MAK) | grep "=y$$" | sort -u
> > $@," GEN $@")
>
> +%/config-devices.mak: default-configs/%.mak
> + $(call quiet-command,cp -p $< $@, " GEN $@")
> +
> -include config-all-devices.mak
>
> build-all: $(DOCS) $(TOOLS) recurse-all
> diff --git a/configure b/configure
> index b65c11c..1223fc8 100755
> --- a/configure
> +++ b/configure
> @@ -2209,10 +2209,6 @@ if test "$target" = "arm-linux-user" -o "$target" =
> "armeb-linux-user" -o "$targ
> mkdir -p $target_dir/nwfpe
> fi
>
> -if test ! -f $target_dir/config-devices.mak ; then
> - cp $source_path/default-configs/${target}.mak
> $target_dir/config-devices.mak
> -fi
> -
> #
> # don't use ln -sf as not all "ln -sf" over write the file/link
> #
> --
> 1.5.6.5
Acked-by Scott Tsai <address@hidden>
I also hit this with the recent change to move two NICs into libhw and
build them for ARM targets.