move bash flag to set statement (#3494)

Passing bash with flags to the first argument of /usr/bin/env requires its own flag to interpret it correctly. Rather than use `env -S` to split the argument, have the script `set -e` to enable the same behavior explicitly in preinst and postinst scripts. Also set `-o pipefail` for consistency. Closes: #3492 Signed-off-by: Cole White <cwhite@wikimedia.org>
opensearch-project · Jun 20, 2022 · 0b4a4f4 · 0b4a4f4
1 parent bd5a004
commit 0b4a4f4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/buildSrc/src/main/resources/deb/postinst.ftl b/buildSrc/src/main/resources/deb/postinst.ftl
@@ -1,2 +1,3 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+set -e -o pipefail
 <% commands.each {command -> %><%= command %><% } %>
diff --git a/buildSrc/src/main/resources/deb/preinst.ftl b/buildSrc/src/main/resources/deb/preinst.ftl
@@ -1,2 +1,3 @@
-#!/usr/bin/env bash -e
+#!/usr/bin/env bash
+set -e -o pipefail
 <% commands.each {command -> %><%= command %><% } %>