Re: [Buildroot] [PATCH v1 3/3] boot/uboot: add host-python-pylibfdt dependency if needed

From: Giulio Benetti <giulio.benetti@benettiengineering.com>
To: Christian Stewart <christian@aperture.us>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: "Yann E . MORIN" <yann.morin.1998@free.fr>,
	Buildroot Mailing List <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH v1 3/3] boot/uboot: add host-python-pylibfdt dependency if needed
Date: Mon, 31 Jul 2023 17:02:18 +0200	[thread overview]
Message-ID: <4df8a509-83da-e0ef-f92e-d708e4d959e6@benettiengineering.com> (raw)
In-Reply-To: <CA+h8R2pz8HTe3DRdSkrh1CwwDy427YWJJBcXN_gh-mPmkPU0WQ@mail.gmail.com>

On 31/07/23 16:56, Christian Stewart via buildroot wrote:
> Thomas,
> 
> On Mon, Jul 31, 2023, 7:43 AM Thomas Petazzoni 
> <thomas.petazzoni@bootlin.com <mailto:thomas.petazzoni@bootlin.com>> wrote:
> 
>     On Mon, 31 Jul 2023 16:10:48 +0200
>     Giulio Benetti <giulio.benetti@benettiengineering.com
>     <mailto:giulio.benetti@benettiengineering.com>> wrote:
> 
>      > U-Boot approach for this is to force Python3 to build libfdtpy[1]
>      > but what we experience here is a consequence of Buildroot's commit
>      > 231d79c81e9a1f8c2ef14861374a40fcdc5e6b33 [2]
>      >
>      > Something happens after that patch has been applied that basically
>      > passes Buildroot's host dtc to UBOOT_MAKE_OPTS:
>      > ```
>      > UBOOT_MAKE_OPTS += DTC=$(HOST_DIR)/bin/dtc
>      > ```
>      > The root cause seems to be Python3 anyway as U-Boot patch points.
>      >
>      > Since we can't patch every U-Boot version we could fix Python3 but
>      > using the approach proposed by Christian is easier to be honest.
>      >
>      > To prove it's a Python3 issue you can see from one of my Gitlab-CI
>      > failure[3] that various U-Boot versions are different, not the last
>      > one(2023.07).
> 
>     Hm, thanks for your feedback, but even after reading 3 times your
>     e-mail, I still don't grasp what's going on. You've pointed out that
>     Buildroot commit 231d79c81e9a1f8c2ef14861374a40fcdc5e6b33 is when build
>     failures starting to occur. But what's the relationship between setting
>     DTC and U-Boot having issues finding pylibfdt?
> 
> 
> If DTC is set the uboot makefile attempts to use the external python 
> package instead of building the internal one

This ^^^ is what I've meant :-)

-- 
CEO/CTO@Benetti Engineering sas
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot