[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] build: get dependency file directories from
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] build: get dependency file directories from object file names |
Date: |
Tue, 24 Jul 2012 14:02:15 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120615 Thunderbird/13.0.1 |
Il 24/07/2012 13:53, Andreas Färber ha scritto:
> Am 24.07.2012 13:43, schrieb Paolo Bonzini:
>> Il 24/07/2012 13:38, Andreas Färber ha scritto:
>>>>> +$(foreach var,$(nested-vars), $(eval \
>>>>> + -include $(addsuffix *.d, $(sort $(dir $($(var)))))))
>>>>> endef
>>>>>
>>> Are you sure? Dependencies are not guaranteed to be in one of the
>>> unnested variables, thought we ran into some issues earlier... In
>>> particular I'm thinking of tcg/*.d and of *.d in .user, .target, those
>>> that are actually recursed into.
>>
>> obj-y is a nested variable:
>>
>> nested-vars += obj-y
>>
>> # This resolves all nested paths, so it must come last
>> include $(SRC_PATH)/Makefile.objs
>>
>> $(dir $(obj-y)) contains tcg/ and from there you get tcg/*.d.
>>
>> The only makefile that doesn't use the nesting mechanism is tests/Makefile.
>
> I think you misunderstood: My worry is
>
> qemu-something: $(obj-y) foo bar baz
>
> obj-y and friends are handled, foo, bar, baz are not. If you checked all
> those cases, fine with me. :) Otherwise the new unnesting rule is fine,
> but not all "*.d" inclusions could be deleted.
Ah, yes, I did. The only cases are in the toplevel Makefile (and the
included tests/Makefile), hence this statement left in that Makefile:
+-include $(wildcard *.d tests/*.d)
Paolo
[Qemu-devel] [PATCH 1/2] build: include qapi-generated/ files in qga/Makefile.objs, Paolo Bonzini, 2012/07/11
Re: [Qemu-devel] [PATCH 0/2] build: get all *.d files, Paolo Bonzini, 2012/07/24
Re: [Qemu-devel] [PATCH 0/2] build: get all *.d files, Blue Swirl, 2012/07/28