From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46877) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V7L7m-0008C1-DC for qemu-devel@nongnu.org; Thu, 08 Aug 2013 03:59:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V7L7f-0004uQ-N1 for qemu-devel@nongnu.org; Thu, 08 Aug 2013 03:59:10 -0400 Received: from mx3-phx2.redhat.com ([209.132.183.24]:58851) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V7L7f-0004rz-EA for qemu-devel@nongnu.org; Thu, 08 Aug 2013 03:59:03 -0400 Date: Thu, 8 Aug 2013 03:59:00 -0400 (EDT) From: Paolo Bonzini Message-ID: <219654540.11185328.1375948740707.JavaMail.root@redhat.com> In-Reply-To: <1375909624-2916-1-git-send-email-sw@weilnetz.de> References: <1375909624-2916-1-git-send-email-sw@weilnetz.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH for 1.6] w32: Add missing version.o to all executables (fix regression) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Weil Cc: Anthony Liguori , qemu-devel ----- Original Message ----- > From: "Stefan Weil" > To: "qemu-devel" > Cc: "Anthony Liguori" , "Paolo Bonzini" , "Stefan Weil" > Sent: Wednesday, August 7, 2013 11:07:04 PM > Subject: [PATCH for 1.6] w32: Add missing version.o to all executables (fix regression) > > QEMU executables for w32, w64 had included meta information built from > version.rc. These rules were changed several times some months ago. > > The latest version added version.o to the tools, but not to the system > emulations. > > This patch adds the meta information to all system emulations again. > > It builds a version.o for each target (which allows different information > for each target in the future). > > I removed the libtool part with version-lobj-y (why was it added?). For libcacard. I think the bug is simply that these two lines: > version-obj-$(CONFIG_WIN32) += version.o > -version-lobj-$(CONFIG_WIN32) += version.lo should be moved to Makefile.objs. Then if you want you can have separate version.o files in 1.7. For 1.6 that's not necessary. Paolo > -Makefile: $(version-obj-y) $(version-lobj-y) > +Makefile: $(version-obj-y) > > ###################################################################### > # Build libraries > diff --git a/Makefile.target b/Makefile.target > index 9a49852..01037fe 100644 > --- a/Makefile.target > +++ b/Makefile.target > @@ -13,6 +13,12 @@ QEMU_CFLAGS += -I.. > -I$(SRC_PATH)/target-$(TARGET_BASE_ARCH) -DNEED_CPU_H > > QEMU_CFLAGS+=-I$(SRC_PATH)/include > > +version.o: $(SRC_PATH)/version.rc ../config-host.h > + > +version-obj-$(CONFIG_WIN32) += version.o > + > +Makefile: $(version-obj-y) > + > ifdef CONFIG_USER_ONLY > # user emulator name > QEMU_PROG=qemu-$(TARGET_NAME) > diff --git a/rules.mak b/rules.mak > index 4499745..0bcd876 100644 > --- a/rules.mak > +++ b/rules.mak > @@ -20,7 +20,7 @@ QEMU_INCLUDES += -I$( %.o: %.c > $(call quiet-command,$(CC) $(QEMU_INCLUDES) $(QEMU_CFLAGS) $(QEMU_DGFLAGS) > $(CFLAGS) -c -o $@ $<," CC $(TARGET_DIR)$@") > %.o: %.rc > - $(call quiet-command,$(WINDRES) -I. -o $@ $<," RC $(TARGET_DIR)$@") > + $(call quiet-command,$(WINDRES) -I $(BUILD_DIR) -o $@ $<," RC > $(TARGET_DIR)$@") > > ifeq ($(LIBTOOL),) > LINK = $(call quiet-command,$(CC) $(QEMU_CFLAGS) $(CFLAGS) $(LDFLAGS) -o $@ > \ > @@ -38,8 +38,7 @@ LIBTOOL += $(if $(V),,--quiet) > LINK = $(call quiet-command,\ > $(if $(filter %.lo %.la,$^),$(LIBTOOL) --mode=link --tag=CC \ > )$(CC) $(QEMU_CFLAGS) $(CFLAGS) $(LDFLAGS) -o $@ \ > - $(sort $(filter %.o, $1)) $(filter-out %.o, $1) \ > - $(if $(filter %.lo %.la,$^),$(version-lobj-y),$(version-obj-y)) \ > + $(sort $(filter %.o, $1)) $(filter-out %.o, $1) $(version-obj-y) \ > $(if $(filter %.lo %.la,$^),$(LIBTOOLFLAGS)) \ > $(LIBS),$(if $(filter %.lo %.la,$^),"lt LINK ", " LINK > ")"$(TARGET_DIR)$@") > endif > -- > 1.7.10.4 > >