Re: [Qemu-devel] [PATCH v2 2/2] make: load only required dependency files.

2015-08-06 Thread Markus Armbruster
Paolo Bonzini writes: > On 05/08/2015 19:29, Victor Kaplansky wrote: >> The old rules.mak loads dependency .d files using include directive >> with file glob pattern "*.d". This breaks the build when build tree has >> remanent *.d files from another build. >> >> This patch fixes this by >> - l

Re: [Qemu-devel] [PATCH v2 2/2] make: load only required dependency files.

2015-08-05 Thread Michael S. Tsirkin
On Wed, Aug 05, 2015 at 09:32:18PM +0300, Michael S. Tsirkin wrote: > On Wed, Aug 05, 2015 at 08:29:33PM +0300, Victor Kaplansky wrote: ... > > diff --git a/rules.mak b/rules.mak > > index 6e35c36..4551b9e 100644 > > --- a/rules.mak > > +++ b/rules.mak > > @@ -368,6 +368,6 @@ define unnest-vars > >

Re: [Qemu-devel] [PATCH v2 2/2] make: load only required dependency files.

2015-08-05 Thread Michael S. Tsirkin
On Wed, Aug 05, 2015 at 08:29:33PM +0300, Victor Kaplansky wrote: > The old rules.mak loads dependency .d files using include directive > with file glob pattern "*.d". This breaks the build when build tree has > remanent *.d files from another build. > > This patch fixes this by > - loading prec

Re: [Qemu-devel] [PATCH v2 2/2] make: load only required dependency files.

2015-08-05 Thread Victor kaplansky
On Wed, Aug 05, 2015 at 07:40:00PM +0200, Paolo Bonzini wrote: > > > On 05/08/2015 19:29, Victor Kaplansky wrote: > > diff --git a/hw/i386/Makefile.objs b/hw/i386/Makefile.objs > > index bd4f147..311d364 100644 > > --- a/hw/i386/Makefile.objs > > +++ b/hw/i386/Makefile.objs > > @@ -10,6 +10,8 @@

Re: [Qemu-devel] [PATCH v2 2/2] make: load only required dependency files.

2015-08-05 Thread Paolo Bonzini
On 05/08/2015 19:29, Victor Kaplansky wrote: > The old rules.mak loads dependency .d files using include directive > with file glob pattern "*.d". This breaks the build when build tree has > remanent *.d files from another build. > > This patch fixes this by > - loading precise list of .d file

[Qemu-devel] [PATCH v2 2/2] make: load only required dependency files.

2015-08-05 Thread Victor Kaplansky
The old rules.mak loads dependency .d files using include directive with file glob pattern "*.d". This breaks the build when build tree has remanent *.d files from another build. This patch fixes this by - loading precise list of .d files made from *.o and *.mo. - specifying explicit list of r