[PATCH] tools: add xxd (from vim)

Daniel Golle daniel at makrotopia.org
Mon Mar 1 06:20:55 EST 2021


Hi Petr,

On Mon, Mar 01, 2021 at 09:12:46AM +0100, Petr Štetiar wrote:
> Daniel Golle <daniel at makrotopia.org> [2021-02-28 19:26:07]:
> 
> Hi Daniel,
> 
> > U-Boot requires xxd to create the default environment from an external file
> > as done in uboot-mediatek.
> 
> it's not essential package for everyone, so why are you forcing everyone to
> download Vim now? Can't you solve it as host dependency in that package?
> 
> > Build xxd (only, not the rest of vim) as part of tools to make sure it is
> > present on the buildhost.
> 
> Why not Emacs? :-) I mean, this Vim tarball is 12M (D'oh!), can't this be
> fixed by xxd from Busybox for example? Or by other means, like including the
> small sources in the tree directly?

Busybox is built for the target, not on the host.

> 
> FYI we're likely going to demand swig[1] soon as host dependency and not
> building it as tool, because it would be support and maintenance nightmare.
> Maybe this could be an option for this case as well?
> 
> 1. https://lists.infradead.org/pipermail/openwrt-devel/2021-February/033654.html

There already is a host build of vim, however, it's on packages repo.
I didn't want to import all of that to the main repo, but just build
xxd, which neither has the dependencies nor the build-complexity of
vim, but it's mery a single C file which didn't change in 20 years.

Hence yes, maybe it's ok to duplicate the source file to avoid
downloading vim on the host system.


> 
> Cheers,
> 
> Petr



More information about the openwrt-devel mailing list