Hi Tom,
On Sat, Feb 22, 2020 at 6:52 AM Tom Rini <tr...@konsulko.com> 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.g...@gmx.de> > > > Cc: Masahiro Yamada <masahi...@kernel.org> > > > Signed-off-by: Tom Rini <tr...@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). > > -- > Tom This is one way to suppress the build error for O= builds. But, please notice this is caused by the re-sync mistake. You partially imported commit db547ef1906400eb34682e43035dd4d81b9fdcfb from Linux. You imported the change in scripts/Kbuild.include but did not the change in scripts/Makefile.lib That's why you needed to add -I$(objtree)/$(obj) all over the places. -- Best Regards Masahiro Yamada