[Buildroot] [PATCH] package/bash: add /bin/bash to /etc/shells

Yann E. MORIN yann.morin.1998 at free.fr
Sat Jan 13 11:25:07 UTC 2018


Thomas, All,

On 2018-01-12 21:21 +0100, Thomas Petazzoni spake thusly:
> On Fri, 12 Jan 2018 18:35:10 +0100, Yann E. MORIN wrote:
> > On 2018-01-12 09:59 +0100, Thomas Petazzoni spake thusly:
> > > On Thu, 11 Jan 2018 18:13:57 +0100, Yann E. MORIN wrote:  
> > > >     grep -E '^/bin/bash$' $(TARGET_DIR)/etc/shells >/dev/null 2>&1 \  
> > > grep -q ?  
> > 
> > -q is not POSIX. Redirecting is guaranteed to always work.
> 
> Knowing that Buildroot only supports Linux hosts, do we care about -q
> not being POSIX ?

Linux host does not mean GNU extensions. For example, Alpine is using
busybox, so not GNU grep...

But I back-pedal on this, it seems that -q has been added to POSIX now:
    http://pubs.opengroup.org/onlinepubs/9699919799/utilities/grep.html

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.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list