[Buildroot] [PATCH 1/4] mkpimage: new host package

[Yann E. MORIN <yann.morin.1998@free.fr>]

Thomas, All,

On 2015-10-19 22:53 +0200, Thomas Petazzoni spake thusly:
> On Mon, 19 Oct 2015 21:04:51 +0200, Maxime Hadjinlian wrote:
> > > +ifndef HOSTGO
> > > +HOSTGO := go
> > > +HOSTGO := $(shell which $(HOSTGO) || type -p $(HOSTGO) || echo go)
> > > +endif
> 
> Ouch, having to depend on Go for such a simple tool is indeed not very
> nice :-/

Or we can eventually look at the Go package that was submitted a while
back, and to which I contributed:
    https://patchwork.ozlabs.org/patch/502222/

;-)

> > Even I don't use this tool anymore, after I reversed the code from Altera's
> > own tool, they released the documentation for the SPL signing and the fine
> > people at Barebox, created a small program (in C) that does exactly the
> > same, which you can find there:
> > http://git.pengutronix.de/?p=barebox.git;a=blob;f=scripts/socfpga_mkimage.c;h=1a7a66d98841e9f52c3ea49c651286aa1412c9a5;hb=HEAD
> > 
> > However, I don't know how you would integrate such a tool, having a special
> > package ? Making U-Boot depends on Barebox sounds a bit crazy. But it would
> > be lighter than having to grab the go compiler for such a simple program.
> 
> I agree. Unfortunately, downloading directly
> http://git.pengutronix.de/?p=barebox.git;a=blob_plain;f=scripts/socfpga_mkimage.c;h=$(MKPIMAGE_VERSION);hb=HEAD
> doesn't work because this URL contains ';' which our download
> infrastructure doesn't like much.
> 
> However, this program is so simple that I would advocate taking the
> makedevs route: simply integrate the program source code itself inside
> Buildroot, in package/mkpimage/.

Yup, agreed.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'