[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] build: pass .d file name to scripts/make_device
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH] build: pass .d file name to scripts/make_device_config.sh, fix makefile target |
Date: |
Thu, 12 Mar 2015 16:24:41 +0100 |
On Thu, Mar 12, 2015 at 04:16:28PM +0100, Paolo Bonzini wrote:
> The .d file name must match exactly what is used in the SUBDIR_DEVICES_MAK_DEP
> variable. Instead of making assumptions in the make_device_config.sh script,
> just pass it in.
>
> Similarly, the makefile target may not match the output file name, because
> Makefile uses a temporary file. Instead of making assumptions on what the
> Makefile does, emit the config-devices.mak file to stdout, and use the
> passed-in destination as the makefile target
>
> Reported-by: Peter Maydell <address@hidden>
> Cc: Michael S. Tsirkin <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> Makefile | 3 ++-
> scripts/make_device_config.sh | 18 ++++++++++--------
> 2 files changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 884b59d..88bce56 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -112,7 +112,8 @@ endif
> -include $(SUBDIR_DEVICES_MAK_DEP)
>
> %/config-devices.mak: default-configs/%.mak
> - $(call quiet-command,$(SHELL) $(SRC_PATH)/scripts/make_device_config.sh
> address@hidden $<, " GEN address@hidden")
> + $(call quiet-command, \
> + $(SHELL) $(SRC_PATH)/scripts/make_device_config.sh $<
> $*-config-devices.mak.d $@ > address@hidden, " GEN address@hidden")
> $(call quiet-command, if test -f $@; then \
> if cmp -s address@hidden $@; then \
Should this compare address@hidden to address@hidden
> mv address@hidden $@; \
> diff --git a/scripts/make_device_config.sh b/scripts/make_device_config.sh
> index 7958086..c1afb3f 100644
> --- a/scripts/make_device_config.sh
> +++ b/scripts/make_device_config.sh
> @@ -1,10 +1,12 @@
> #! /bin/sh
> -# Construct a target device config file from a default, pulling in any
> -# files from include directives.
> +# Writes a target device config file to stdout, from a default and from
> +# include directives therein. Also emits Makefile dependencies.
> +#
> +# Usage: make_device_config.sh SRC DEPFILE-NAME DEPFILE-TARGET > DEST
>
> -dest=$1
> -dep=`dirname $1`-`basename $1`.d
> -src=$2
> +src=$1
> +dep=$2
> +target=$3
> src_dir=`dirname $src`
> all_includes=
>
> @@ -22,7 +24,7 @@ while [ -n "$f" ] ; do
> [ $? = 0 ] || exit 1
> all_includes="$all_includes $f"
> done
> -process_includes $src > $dest
> +process_includes $src
>
> -cat $src $all_includes | grep -v '^include' > $dest
> -echo "$1: $all_includes" > $dep
> +cat $src $all_includes | grep -v '^include'
> +echo "$target: $all_includes" > $dep
> --
> 2.3.0