On Thu, Jan 19, 2023 at 11:52:11AM +0000, Peter Robinson wrote: > On Thu, Jan 19, 2023 at 11:33 AM Fabio Estevam <feste...@gmail.com> wrote: > > > > On Thu, Jan 19, 2023 at 8:18 AM Peter Robinson <pbrobin...@gmail.com> wrote: > > > > > Did you read the original thread? > > > > I only read your commit log and it is not clear if it is a regression > > and which commit caused the problem. > > I don't remember, this was back in November. > > > It lacks a Fixes tag too. > > Which are mostly pointless in U-Boot as it's not like the kernel where > there's stable release cycles.
They're quite helpful, in order to track when changes are fixing a
problem that was introduced by some other commit.
> > > > - What is the exact problem you are trying to solve? How can we
> > > > reproduce it?
> > >
> > > Building that tool with the tools only option.
> >
> > When I try to build U-Boot 2023.01 u-boot-tools-native in OpenEmbedded
> > it works fine.
>
> But does it actually build the bmp_logo tool? It builds fine for me
> too but that tool is no longer built.
>
> > It fails when trying to build it for the target since commit
> > 1cfba53ca46c ("config: tools only: add VIDEO to build
> > bmp_logo").
>
> Where when I build it for Fedora it builds for me and I get the
> bmp_logo which we used to have pre 2023.01
>
> > > > - Is it a regression? What is the commit that caused the problem you
> > > > are trying to solve?
> >
> > What about this part?
>
> I don't remember, see comment above.
The unanswered question is, why does Fedora ship the bmp_logo tool? I'm
fairly certain it's one of the host tools that's not useful outside of
the build and so shouldn't be packaged.
--
Tom
signature.asc
Description: PGP signature
