[Buildroot] [PATCH] boot/uboot: fix uboot build
Köry Maincent
kory.maincent at bootlin.com
Tue Jun 29 12:47:35 UTC 2021
Hello Yann,
On Mon, 28 Jun 2021 22:26:56 +0200
"Yann E. MORIN" <yann.morin.1998 at free.fr> wrote:
> 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? ;-)
Sorry, I put it aside for a time.
I need to finish the test case asked by Thomas, I will try to find time to do
it soon.
Regards
Köry
More information about the buildroot
mailing list