[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