[U-Boot] [imx/next V2] tools: imx8m_image: use bash

Peng Fan peng.fan at nxp.com
Thu Dec 13 07:15:04 UTC 2018


Hi Jon

> -----Original Message-----
> From: Jon Nettleton [mailto:jon at solid-run.com]
> Sent: 2018年12月13日 14:54
> To: Peng Fan <peng.fan at nxp.com>
> Cc: sbabic at denx.de; Fabio Estevam <fabio.estevam at nxp.com>; U-Boot
> Mailing List <u-boot at lists.denx.de>
> Subject: Re: [U-Boot] [imx/next V2] tools: imx8m_image: use bash
> 
> On Wed, Dec 12, 2018 at 2:54 AM Peng Fan <peng.fan at nxp.com> wrote:
> >
> > travis-ci reports
> > "tools/imx8m_image.sh: 15: [: signed_hdmi_imx8m.bin: unexpected
> operator"
> > After use bash, no error, so use bash instead of sh.
> >
> > Signed-off-by: Peng Fan <peng.fan at nxp.com>
> > ---
> >  tools/imx8m_image.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tools/imx8m_image.sh b/tools/imx8m_image.sh index
> > 6346fb64d8..0e9df4763c 100755
> > --- a/tools/imx8m_image.sh
> > +++ b/tools/imx8m_image.sh
> > @@ -1,4 +1,4 @@
> > -#!/bin/sh
> > +#!/bin/bash
> >  # SPDX-License-Identifier: GPL-2.0+
> >  #
> >  # script to check whether the file exists in imximage.cfg for i.MX8M
> 
> We have a patch that fixes this to work with sh, dash, and bash if you prefer.

Sure, please submit a patch based on imx/next, I would like those patches could be in master soon.

Thanks,
Peng.


More information about the U-Boot mailing list