From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41598) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZNGzA-0004ES-Vf for qemu-devel@nongnu.org; Thu, 06 Aug 2015 04:57:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZNGz5-0000hl-6q for qemu-devel@nongnu.org; Thu, 06 Aug 2015 04:57:12 -0400 Received: from mail-wi0-f172.google.com ([209.85.212.172]:35913) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZNGz5-0000hO-26 for qemu-devel@nongnu.org; Thu, 06 Aug 2015 04:57:07 -0400 Received: by wicgj17 with SMTP id gj17so13385989wic.1 for ; Thu, 06 Aug 2015 01:57:06 -0700 (PDT) References: <1438795570-3930-1-git-send-email-victork@redhat.com> <1438795570-3930-2-git-send-email-victork@redhat.com> From: Alex =?utf-8?Q?Benn=C3=A9e?= In-reply-to: <1438795570-3930-2-git-send-email-victork@redhat.com> Date: Thu, 06 Aug 2015 09:57:03 +0100 Message-ID: <87vbcskd5s.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH v2 1/2] make: fix where dependency *.d are stored. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Victor Kaplansky Cc: Peter Maydell , Eduardo Habkost , "Michael S. Tsirkin" , Stefan Weil , qemu-devel@nongnu.org, Paolo Bonzini , Igor Mammedov , Richard Henderson Victor Kaplansky writes: > In rules like "bar/%.o: %.c" there is a difference between $(*D) and > $(@D). It is cleaner to generate *.d next to appropriate *.o, because it > allows precise including of dependency info from .d files. It might be worth putting the example in the comment as most people are not familiar with the esoteric symbolic of GNU Make. AIUI in your example $(@D) resolves to "bar" instead of "bar/%" > > As a hack, we also touch two sources for generated *.hex files. This is > to ensure *.hex rebuild, when old revision is switched to the new one. While I'm sure this was useful for testing do we really want to include such a hack in the git history? > > Signed-off-by: Victor Kaplansky > --- > hw/i386/acpi-dsdt.dsl | 1 - > hw/i386/q35-acpi-dsdt.dsl | 1 + > rules.mak | 2 +- > 3 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/i386/acpi-dsdt.dsl b/hw/i386/acpi-dsdt.dsl > index a2d84ec..8dba096 100644 > --- a/hw/i386/acpi-dsdt.dsl > +++ b/hw/i386/acpi-dsdt.dsl > @@ -43,7 +43,6 @@ DefinitionBlock ( > > #include "acpi-dsdt-hpet.dsl" > > - > /**************************************************************** > * PIIX4 PM > ****************************************************************/ > diff --git a/hw/i386/q35-acpi-dsdt.dsl b/hw/i386/q35-acpi-dsdt.dsl > index 16eaca3..7be7b37 100644 > --- a/hw/i386/q35-acpi-dsdt.dsl > +++ b/hw/i386/q35-acpi-dsdt.dsl > @@ -22,6 +22,7 @@ > * Based on acpi-dsdt.dsl, but heavily modified for q35 chipset. > */ > > + > ACPI_EXTRACT_ALL_CODE Q35AcpiDsdtAmlCode > > DefinitionBlock ( > diff --git a/rules.mak b/rules.mak > index aec27f8..6e35c36 100644 > --- a/rules.mak > +++ b/rules.mak > @@ -17,7 +17,7 @@ MAKEFLAGS += -rR > QEMU_CXXFLAGS = -D__STDC_LIMIT_MACROS $(filter-out -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs -Wold-style-declaration -Wold-style-definition -Wredundant-decls, $(QEMU_CFLAGS)) > > # Flags for dependency generation > -QEMU_DGFLAGS += -MMD -MP -MT $@ -MF $(*D)/$(*F).d > +QEMU_DGFLAGS += -MMD -MP -MT $@ -MF $(@D)/$(*F).d > > # Same as -I$(SRC_PATH) -I., but for the nested source/object directories > QEMU_INCLUDES += -I$(