[PATCH] efi_selftest: Ensure we include the object directory for generated headers

Heinrich Schuchardt xypron.glpk at gmx.de
Fri Feb 21 23:56:14 CET 2020


On 2/21/20 10:52 PM, Tom Rini wrote:
> On Fri, Feb 21, 2020 at 10:34:01PM +0100, Heinrich Schuchardt wrote:
>> On 2/21/20 9:48 PM, Tom Rini wrote:
>>> The current rules for generating -I lines for objects gives us both
>>> -I/full/src/path/lib/efi_selftest and -Ilib/efi_selftest.  However, if
>>> we re-sync the Kbuild logic we no longer get the latter implicitly.  We
>>> will need to be explicit in such cases, so do so.
>>>
>>> Cc: Heinrich Schuchardt <xypron.glpk at gmx.de>
>>> Cc: Masahiro Yamada <masahiroy at kernel.org>
>>> Signed-off-by: Tom Rini <trini at konsulko.com>
>>
>> Hello Tom,
>>
>> yes, in lib/efi_selftest we have generated includes, so
>> -Ilib/efi_selftest is relevant.
>>
>> Do you have a git branch with the Kconfig changes for reference and testing?
>
> Currently WIP/kbuild-resync-v1-v4.18 but please bear in mind I'm (a)
> rebase it and (b) it doesn't even build sandbox_spl right now (more of
> this type of error, but my non-CI world build will finish up shortly and
> I'll grab all of the places I need to make a fix like this there).
>

I was able to reproduce that the proposed change for
lib/efi_selftest/Makefile is needed for out of tree building.

So the patch is fine with me.

Best regards

Heinrich


More information about the U-Boot mailing list