[Buildroot] [PATCH 01/69] am335x-pru-package: add explicit dependencies for host variant

Julien Floret julien.floret at 6wind.com
Sun Jul 3 07:26:56 UTC 2016


Hello Thomas,

2016-07-03 8:32 GMT+02:00 Thomas Petazzoni
<thomas.petazzoni at free-electrons.com>:
> Hello,
>
> On Sun,  3 Jul 2016 00:20:30 +0200, Julien Floret wrote:
>> Signed-off-by: Julien Floret <julien.floret at 6wind.com>
>> ---
>>  package/am335x-pru-package/am335x-pru-package.mk | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/package/am335x-pru-package/am335x-pru-package.mk b/package/am335x-pru-package/am335x-pru-package.mk
>> index a97e8a49eacb..8b33eb3d6cbf 100644
>> --- a/package/am335x-pru-package/am335x-pru-package.mk
>> +++ b/package/am335x-pru-package/am335x-pru-package.mk
>> @@ -9,6 +9,7 @@ AM335X_PRU_PACKAGE_SITE = $(call github,beagleboard,am335x_pru_package,$(AM335X_
>>  AM335X_PRU_PACKAGE_LICENSE = BSD-3c
>>  AM335X_PRU_PACKAGE_LICENSE_FILES = pru_sw/utils/LICENCE.txt
>>  AM335X_PRU_PACKAGE_DEPENDENCIES = host-am335x-pru-package
>> +HOST_AM335X_PRU_PACKAGE_DEPENDENCIES = host-am335x-pru-package
>
> With this addition, host-am335x-pru-package now depends on
> host-am335x-pru-package, which introduces a circular dependency.
>
> Having:
>
>   AM335X_PRU_PACKAGE_DEPENDENCIES = host-am335x-pru-package
>
> does *not* imply:
>
>   HOST_AM335X_PRU_PACKAGE_DEPENDENCIES = host-am335x-pru-package
>
> Because in the logic to derive host dependencies from target
> dependencies, we filter out if the package would depend on itself:
>
> $(2)_DEPENDENCIES ?= $$(filter-out host-skeleton host-toolchain $(1),\
>         $$(patsubst host-host-%,host-%,$$(addprefix host-,$$($(3)_DEPENDENCIES))))
>
> See the $(1) in the call of filter-out ?
>
> I'll mark this patch as Rejected then.

Of course. I stupidly missed this one, as well as in the "file" package.
Thanks,

Julien


More information about the buildroot mailing list