[Buildroot] [git commit branch/2019.11.x] package/grep: fix egrep/fgrep shebang

Peter Korsgaard peter at korsgaard.com
Tue Mar 10 20:31:44 UTC 2020

commit: https://git.buildroot.net/buildroot/commit/?id=ab35d7d176547de331f82c3f186db81d36316ab9
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/2019.11.x

egrep/fgrep are wrapper scripts, calling the grep binary with the correct

The shell wrappers use the value of SHELL at build time as the shebang value
in these wrapper scripts, which in Buildroot points to /bin/bash.

The target may not have bash available, causing runtime errors.

As a fix, add a post-install hook to change this to /bin/sh.

If the target does not have /bin/sh, simply remove the aliases.

Signed-off-by: Angelo Compagnucci <angelo at amarulasolutions.com>
Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
(cherry picked from commit 8f9f48acd2db27863ca8fc0a4f1377aa2b07c886)
Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
 package/grep/Config.in |  2 ++
 package/grep/grep.mk   | 17 +++++++++++++++++
 2 files changed, 19 insertions(+)

diff --git a/package/grep/Config.in b/package/grep/Config.in
index 5b0471b4c0..665c05c05e 100644
--- a/package/grep/Config.in
+++ b/package/grep/Config.in
@@ -5,6 +5,8 @@ config BR2_PACKAGE_GREP
 	  The GNU regular expression matcher.
+	  egrep/fgrep aliases need /bin/sh to be available.
 comment "grep needs a toolchain w/ wchar"
diff --git a/package/grep/grep.mk b/package/grep/grep.mk
index eefdfe6a74..cc80553eb9 100644
--- a/package/grep/grep.mk
+++ b/package/grep/grep.mk
 # install into /bin like busybox grep
 GREP_CONF_OPTS = --exec-prefix=/
+ifeq ($(BR2_SYSTEM_BIN_SH_NONE),y)
+	$(RM) $(TARGET_DIR)/bin/[fe]grep
+# ensure egrep/fgrep shell wrappers use #!/bin/sh
+	$(SED) 's/bash$$/sh/' $(TARGET_DIR)/bin/[fe]grep
 # link with iconv if enabled
 GREP_CONF_ENV += LIBS=-liconv

More information about the buildroot mailing list