[OE-core] [PATCH v7] u-boot: Add mkenvimage tool

[Tom Rini]

On Mon, Nov 26, 2018 at 09:07:37PM -0200, Otavio Salvador wrote:
> Hello Alexey,
> 
> On Mon, Nov 26, 2018 at 7:30 PM Alexey Brodkin
> <alexey.brodkin at synopsys.com> wrote:
> > On Sat, 2018-11-24 at 06:57 -0800, Khem Raj wrote:
> > >
> > > http://errors.yoctoproject.org/Errors/Details/202185/
> >
> > Looking at the target name "u-boot-tools-1_2018.11-r0 do_compile"
> > I think it might have something to do with bump of U-Boot to v2018.11
> > and the error message in question was added in v2018.11-rc1,
> > see http://git.denx.de/?p=u-boot.git;a=commitdiff;h=a4958a71017fb142542f977c843c5fce769fc6ea
> >
> > The problem is we use "sandbox_defconfig" as a dummy defconfig
> > because otherwise (not configured) U-Boot's build-system won't
> > allow us to proceed and in its turn mentioned patch adds a check
> > for a target platform like "if defined(__x86_64__)" which
> > we essentially don't satisfy and so we fail.
> >
> > There're the following solutions:
> >  1. Revert mentioned patch
> >  2. Disable "CONFIG_EFI_LOADER"
> >
> > Reverting is not nice and not future-proof.
> > Disabling of "CONFIG_EFI_LOADER" might be done in a couple of ways
> > like:
> >  1. Patching sandbox_defconfig
> >  2. Filtering "CONFIG_EFI_LOADER" from "sandbox_defconfig" right before
> >     execution of "make sandbox_defconfig"
> >
> > I think latter option is the simplest and cleanest.
> >
> > Should I send a patch for that?
> 
> Let's add Tom (U-Boot upstream main maintainer) to this so he can comment.

Hey, so you've hit the problem Fedora also hit, but we haven't gotten a
real solution to.  Yes, the best "fix" here for just building tools
would be to turn off EFI_LOADER in sandbox, on all arches.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20181126/42209f20/attachment.sig>