Re: [OE-core] [PATCH v4 2/7] barebox-tools: add initial barebox tools support

Enrico Jörns
2024-09-10T09:02:51Z

Am Dienstag, dem 10.09.2024 um 10:49 +0200 schrieb Alexander Kanavin:
> On Tue, 10 Sept 2024 at 10:45, Enrico Jörns <ejo@pengutronix.de> wrote:
> > > Can this be merged into the main barebox recipe? Does it need a separate recipe?
> >
> > not only that it would complicate the main recipe (we had tried this before), but I really see
> > no
> > point in doing and supporting barebox native builds (in contrast to barebox-tools).
> >
> > And some of these tools might also be useful without using barebox.
> >
> > I guess u-boot-tools is separate for similar reasons.
>
> Then you should put the common bits into barebox-common.inc, same as
> u-boot does.

My impression was that the actual amount of things to share would not justify another file.
It is mainly license information, SRC_URI and a bit of metadata where I found it more obvious to
have them in the respective recipe.

If having shared parts in a barebox-common.inc is actually preferable, I could rework this of
course.

> Also there should be a good DESCRIPTION in there. Why
> barebox, and not u-boot, for example?

Trying to outdo other projects was not my intention.
So I would like to omit such comparisons from the description.

My current summary:

"barebox is a bootloader designed for embedded systems.
It runs on a variety of architectures including x86, ARM, MIPS, PowerPC and others."

does not yet get the point of a bootloader?
I also thought that adding too much information would make the summary too long.
But in case something important is missing, I am open to further suggestions.

Thanks in advance an best regards

Enrico

> Alex
>

--
Pengutronix e.K.                           | Enrico Jörns                |
Embedded Linux Consulting & Support        | https://www.pengutronix.de/ |
Steuerwalder Str. 21                       | Phone: +49-5121-206917-180  |
31137 Hildesheim, Germany                  | Fax:   +49-5121-206917-9    |