move bash flag to set statement (opensearch-project#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: opensearch-project#3492 Signed-off-by: Cole White <cwhite@wikimedia.org>
Bukhtawar · Jun 21, 2022 · e10aadc · e10aadc
1 parent 2564cab
commit e10aadc
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 %><% } %>