[Buildroot] [PATCH] boot/uboot: fix uboot build
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Jun 28 20:26:56 UTC 2021
Köry, All,
On 2021-03-25 17:34 +0100, Köry Maincent spake thusly:
> On Tue, 16 Mar 2021 23:08:52 +0100
> Thomas Petazzoni <thomas.petazzoni at bootlin.com> wrote:
> > On Thu, 4 Feb 2021 18:04:46 +0100
> > Kory Maincent <kory.maincent at bootlin.com> wrote:
> > > The make all command run the tools/makefile on the process.
> > > This makefile use "pkg-config" command to support static link.
> > > The issue is the use of pkg-config configured for crosscompiling
> > > to build binaries tools for host architecture.
> > > To fix it, I copied the behavior of QtWebengine by using a pkg-config
> > > executable preconfigured for host.
> > >
> > > Signed-off-by: Kory Maincent <kory.maincent at bootlin.com>
> > Thanks for the patch. It would require a better commit title though, as
> > "fix uboot build" is very vague.
> Ok I will change it for v2.
> > After discussing with Yann E. Morin on IRC, could you try this change
> > instead:
> >
> > $(TARGET_CONFIGURE_OPTS) \
> > PKG_CONFIG="$(PKG_CONFIG_HOST_BINARY)" \
> > PKG_CONFIG_SYSROOT_DIR="/" \
> > PKG_CONFIG_ALLOW_SYSTEM_CFLAGS=1 \
> > PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 \
> > PKG_CONFIG_LIBDIR="$(HOST_DIR)/lib/pkgconfig:$(HOST_DIR)/share/pkgconfig"
> >
> > The reasoning is that U-Boot anyway only uses pkg-config for host
> > tools, so we could just as well pass those environment variables when
> > building U-Boot.
> >
> > Could you try this out?
>
> This change works well, the build of U-boot ends properly.
I don't seem to have seen a v2 of this patch. Do you plan on sending it
soonish? ;-)
Thanks!
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list