HomeFreeBSD

dracut: use /bin/sh instead of bash as the intepreter

Description

dracut: use /bin/sh instead of bash as the intepreter

Despite that dracut has a hard dependency on bash,
its modules doesn't, dracut only has a hard dependency on bash for
module-setup (on a fully usable machine). Inside initramfs, dracut
allows users choose from a list of handful other shells, e.g. bash,
busybox, dash, mkfsh.

In fact, my local machine's initramfs is being built with dash,
and it's functional for a very long time.

Before 64025fa3a (Silence 'make checkbashisms', 2020-08-20), we also
allows our users to have that right, too.

Let's fix the problem 'make checkbashisms' reported and allows our users
to have that right, again.

For 'plymouth' case, let's simply run the command inside the if instead
of checking for the existence of command before running it, because the
status is also failture if plymouth is unavailable.

While we're at it, let's remove an unnecessary fork for grep in
zfs-generator.sh.in and its following complicated 'if elif fi' with
a simple 'case ... esac'.

To support this change, also exclude 90zfs from "make checkbashisms"
because the current CI infrastructure ships an old version of
"checkbashisms", which complains about "command -v", while the current
latest "checkbashisms" thinks it's fine. In the near future, we can
revert that change to "Makefile.am" when CI infrastructure is updated.

Reviewed-by: Gabriel A. Devenyi <gdevenyi@gmail.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com>
Closes #11244

Details

Provenance
Đoàn Trần Công Danh <congdanhqx@gmail.com>Authored on Nov 28 2020, 7:02 PM
GitHub <noreply@github.com>Committed on Nov 28 2020, 7:02 PM
Parents
rG04a82e043d06: Remove incorrect assertion
Branches
Unknown
Tags
Unknown

Event Timeline

GitHub <noreply@github.com> committed rG16692e6ba0bc: dracut: use /bin/sh instead of bash as the intepreter (authored by Đoàn Trần Công Danh <congdanhqx@gmail.com>).Nov 28 2020, 7:02 PM