[Buildroot] [PATCH v2 2/3] package: fix generic extract target for top-level parallel make
Fabio Porcedda
fabio.porcedda at gmail.com
Mon Aug 26 08:29:52 UTC 2013
On Thu, Aug 22, 2013 at 5:59 PM, Arnout Vandecappelle <arnout at mind.be> wrote:
> On 22/08/13 09:44, Fabio Porcedda wrote:
>>
>> On Wed, Aug 21, 2013 at 9:24 PM, Arnout Vandecappelle <arnout at mind.be>
>> wrote:
>>>
>>> On 18/07/13 11:12, Fabio Porcedda wrote:
>>>>
>>>>
> [snip]
>
>>>>
>>>> $(1)-patch: $(1)-extract $$($(2)_TARGET_PATCH)
>>>>
>>>> -$(1)-extract: $(1)-source \
>>>> - $$($(2)_TARGET_EXTRACT)
>>>> +$$($(2)_TARGET_EXTRACT): $$($(2)_TARGET_SOURCE)
>>>> +$(1)-extract: $$($(2)_TARGET_EXTRACT)
>>>>
>>>> $(1)-depends: $$($(2)_DEPENDENCIES)
>>>>
>>>>
>>>
>>> On second observation, I don't really like this change, because it
>>> makes
>>> the extract and patch parts asymmetrical with the others. I would prefer
>>> one
>>> patch that changes it for all the targets. But then, the behaviour does
>>> change, because rebuilding one package will also trigger a rebuild of the
>>> packages that depend on it. Which may be a good thing, of course...
>>
>>
>> Do you mean a single patch that changes all the targets? IMHO the
>> patch becomes too complex, but if is the preferred way i'm fine with
>> that.
>
>
> Yes. I estimate it will modify about 50 lines, so I don't see a problem.
>
>
>>
>> To be able to change the others targets i need to add stamp file for
>> every target inside $$($(2)_DEPENDENCIES,
>> i need to do that because a file cannot depends on a non existing file.
>
>
> That's not true. Take the following Makefile:
>
> ----------------
> %.source:
> touch $@
>
> %.extract: %.source
> touch $@
>
> %.config: %.extract
> touch $@
>
> %.build: %.config
> touch $@
>
> X_DEPS = y z
>
> x.config: $(X_DEPS)
The problem with this solution is that the "x.config" target is
reevaluated every time because it does not depends on a virtual
target, the only solution that i found is to add a stamp file for
every dependencies.
Using virtual target as dependency is fine only with virtual targets.
Regards
--
Fabio Porcedda
More information about the buildroot
mailing list