From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55058) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YW4yh-0002oJ-HY for qemu-devel@nongnu.org; Thu, 12 Mar 2015 11:24:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YW4yc-00006x-Dt for qemu-devel@nongnu.org; Thu, 12 Mar 2015 11:24:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:35934) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YW4yc-00006g-6M for qemu-devel@nongnu.org; Thu, 12 Mar 2015 11:24:46 -0400 Date: Thu, 12 Mar 2015 16:24:41 +0100 From: "Michael S. Tsirkin" Message-ID: <20150312162331-mutt-send-email-mst@redhat.com> References: <1426173388-11796-1-git-send-email-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1426173388-11796-1-git-send-email-pbonzini@redhat.com> Subject: Re: [Qemu-devel] [PATCH] build: pass .d file name to scripts/make_device_config.sh, fix makefile target List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org 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 > Cc: Michael S. Tsirkin > Signed-off-by: Paolo Bonzini > --- > 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 $@.tmp $<, " GEN $@.tmp") > + $(call quiet-command, \ > + $(SHELL) $(SRC_PATH)/scripts/make_device_config.sh $< $*-config-devices.mak.d $@ > $@.tmp, " GEN $@.tmp") > $(call quiet-command, if test -f $@; then \ > if cmp -s $@.old $@; then \ Should this compare $@.old to $@.tmp? > mv $@.tmp $@; \ > 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