[Qemu-devel] Re: [PATCH 0/2] Build fixes

[Juan Quintela]

Daniel Jacobowitz <address@hidden> wrote:
> On Thu, Nov 05, 2009 at 02:42:51PM +0100, Juan Quintela wrote:
>> Hi
>> 
>> This series fixes the parallel build of qemu.  I tried to get the best of
>> Daniel and Stefan ideas with mine.
>> Once there fixed an obvious fix.
>> 
>> Daniel, can you check that parallel build also works for you?  It works for 
>> me (TM).
>
> No, it doesn't work.
>
>   GEN  config-all-devices.mak
>   GEN   config-host.h
>   GEN   config-all-devices.h
>   CC    qemu-tool.o
>   CC    cutils.o
> In file included from /scratch/dan/qemu/src/qemu-trunk/qemu-tool.c:14:
> /scratch/dan/qemu/src/qemu-trunk/qemu-common.h:32:25: error:
> config-host.h: No such file or directory
> make: *** [qemu-tool.o] Error 1

How are you compiling?
It works for me compiling in-tree with make -j3 (only 2 cores)

> You have to define GENERATED_HEADERS before you use it in
> dependencies, I believe.  But you've got it way down below the
> rules.mak invocation.

Will try it.

> Also, your changes to $(QEMU_PROG) and recurse-all are not necessary;

before recurse-all is necesary, because the sub-targets use
config-host.h (they don't use config-all-devices.h, but it don't hurt to
also generate it).

> they're the same sort of ordering assumption that caused the original
> bug.  You only need to depend on generated files if you're going to
> use them in this rule's commands; for headers, that means invoke a .c
> to .o compilation.  The dependencies should be on exactly the rules
> that need them.
>
> I didn't bother with Makefile.target since the headers are generated
> in the top level and that make will remain recursive.

Not all of them.
configure-target.h and config-device.h is generated there.

</me studies still more the Makefiles>

ok, subdir-% already depends on them.

I am sending a new patch with you suggestions, it works for me (again).

on-tree and out-of-tree compilation with make -j (still on my laptop
with 2 cores).

Can you test my new series?

Later, Juan.