Re: [PATCH 1/1] toolchain-shar-relocate.sh: Add error-handling

[Patrick Ohly <patrick.ohly@intel.com>]

On Fri, 2016-09-23 at 15:48 +0100, Richard Purdie wrote:
> On Wed, 2016-09-21 at 19:06 -0700, Christopher Larson wrote:
> > 
> > On Wed, Sep 21, 2016 at 12:54 AM, <mariano.lopez@linux.intel.com>
> > wrote:
> > > diff --git a/meta/files/toolchain-shar-relocate.sh
> > > b/meta/files/toolchain-shar-relocate.sh
> > > index d4bcf0e..e491153 100644
> > > --- a/meta/files/toolchain-shar-relocate.sh
> > > +++ b/meta/files/toolchain-shar-relocate.sh
> > > @@ -1,3 +1,8 @@
> > > +if ! xargs --version > /dev/null 2>&1; then
> > > +       echo "xargs is required by the relocation script, please
> > > install it firts. Abort!"
> > > +       exit 1
> > > +fi
> > > 
> > This is pointless, xargs is mandated by posix/sus.
> 
> It appears we have some users with stripped down containers where it
> hasn't been installed though and having a user readable message is
> better

... and even better is one without typos ;-}

s/firts/first/

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.