On 2023-01-13, Tom Rini wrote:
> On Fri, Jan 13, 2023 at 06:26:00PM -0300, Fabio Estevam wrote:
>> I am trying to upgrade U-Boot to 2023.01 in OpenEmbedded, but I see
>> the following error when trying to build u-boot-tools:
>> 
>> | /bin/sh: line 1: tools/bmp_logo: No such file or directory
>> 
>> Reverting the commit below makes u-boot-tools build again:
>> 
>> commit 1cfba53ca46cade2dbf4e067afc8c19e72909a4b
>> Author: Peter Robinson <pbrobin...@gmail.com>
>> Date:   Thu Nov 24 14:05:59 2022 +0000
>> 
>>     config: tools only: add VIDEO to build bmp_logo
>> 
>>     Pre 2023.01 the bmp_logo was built as part of the tools-only_defconfig
>>     build, something changed and the VIDEO dep needed to build it
>>     is no longer pulled in so fix that by explicitly defining it.
>> 
>>     Signed-off-by: Peter Robinson <pbrobin...@gmail.com>
>>     Reviewed-by: Simon Glass <s...@chromium.org>
>> 
>> u-boot-tools-native builds fine though.
>> 
>> What would be the correct way to fix this?
>
> Vagrant also hit this for Debian as Fedora cross-builds bmp_logo and
> ships it in host tools, but the logo header files that we generate with
> bmp_logo need the tool host built.  And the same flag, VIDEO_LOGO
> (default y if VIDEO, basically) controls both the tool and the headers.
> I think the first thing to figure out is if bmp_logo should be shipped
> as a host tool, or not.

FWIW, we have not shipped bmp_logo in Debian's u-boot-tools package, so
we have managed without it, though curious what the use-case might be.

It is disabled in Debian for now:

  
https://salsa.debian.org/debian/u-boot/-/blob/debian/2023.01+dfsg-1/debian/patches/tools-disable-video-logo

... at least until a better fix comes along. :)

live well,
  vagrant

Attachment: signature.asc
Description: PGP signature

Reply via email to