scripted-diff: Use long form of shell options in Guix scripts
What changed, and why it matters
This commit is a purely cosmetic change in Bitcoin Core's Guix build scripts. It replaces the short shell command 'set -e' with its long-form equivalent 'set -o errexit'. Both forms do exactly the same thing: they make the script exit immediately if any command fails. There is no security issue and no functional change.
No action required. This is a readability/style refactor with no security or functional impact.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit applies a scripted sed replacement across six contrib/guix shell scripts, changing ‘set -e -o pipefail’ to ‘set -o errexit -o pipefail’. The -e and -o errexit options are synonyms in bash; the behavior of the scripts is unchanged. No logic, permissions, dependencies, or build outputs are modified.
Changed components
contrib/guix/guix-attestcontrib/guix/guix-cleancontrib/guix/guix-codesigncontrib/guix/guix-verifycontrib/guix/libexec/codesign.shcontrib/guix/libexec/package.shInspect captured patch +6 / −6
diff --git a/contrib/guix/guix-attest b/contrib/guix/guix-attest
index b3a20100..06925ffb 100755
--- a/contrib/guix/guix-attest
+++ b/contrib/guix/guix-attest
@@ -3,7 +3,7 @@
# Distributed under the MIT software license, see the accompanying
# file COPYING or https://opensource.org/license/mit.
export LC_ALL=C.UTF-8
-set -e -o pipefail
+set -o errexit -o pipefail
# Source the common prelude, which:
# 1. Checks if we're at the top directory of the Bitcoin Core repository
diff --git a/contrib/guix/guix-clean b/contrib/guix/guix-clean
index 96cc2832..a72a2644 100755
--- a/contrib/guix/guix-clean
+++ b/contrib/guix/guix-clean
@@ -3,7 +3,7 @@
# Distributed under the MIT software license, see the accompanying
# file COPYING or https://opensource.org/license/mit.
export LC_ALL=C.UTF-8
-set -e -o pipefail
+set -o errexit -o pipefail
# Source the common prelude, which:
# 1. Checks if we're at the top directory of the Bitcoin Core repository
diff --git a/contrib/guix/guix-codesign b/contrib/guix/guix-codesign
index 3b139e1f..179041c1 100755
--- a/contrib/guix/guix-codesign
+++ b/contrib/guix/guix-codesign
@@ -3,7 +3,7 @@
# Distributed under the MIT software license, see the accompanying
# file COPYING or https://opensource.org/license/mit.
export LC_ALL=C.UTF-8
-set -e -o pipefail
+set -o errexit -o pipefail
# Source the common prelude, which:
# 1. Checks if we're at the top directory of the Bitcoin Core repository
diff --git a/contrib/guix/guix-verify b/contrib/guix/guix-verify
index 8e7fbb90..82e74003 100755
--- a/contrib/guix/guix-verify
+++ b/contrib/guix/guix-verify
@@ -3,7 +3,7 @@
# Distributed under the MIT software license, see the accompanying
# file COPYING or https://opensource.org/license/mit.
export LC_ALL=C.UTF-8
-set -e -o pipefail
+set -o errexit -o pipefail
# Source the common prelude, which:
# 1. Checks if we're at the top directory of the Bitcoin Core repository
diff --git a/contrib/guix/libexec/codesign.sh b/contrib/guix/libexec/codesign.sh
index 620fe319..21f4c110 100755
--- a/contrib/guix/libexec/codesign.sh
+++ b/contrib/guix/libexec/codesign.sh
@@ -3,7 +3,7 @@
# Distributed under the MIT software license, see the accompanying
# file COPYING or https://opensource.org/license/mit.
export LC_ALL=C.UTF-8
-set -e -o pipefail
+set -o errexit -o pipefail
# Environment variables for determinism
export TAR_OPTIONS="--owner=0 --group=0 --numeric-owner --mtime='@${SOURCE_DATE_EPOCH}' --sort=name"
diff --git a/contrib/guix/libexec/package.sh b/contrib/guix/libexec/package.sh
index fdecd9a0..05f5e61f 100755
--- a/contrib/guix/libexec/package.sh
+++ b/contrib/guix/libexec/package.sh
@@ -3,7 +3,7 @@
# Distributed under the MIT software license, see the accompanying
# file COPYING or https://opensource.org/license/mit.
export LC_ALL=C.UTF-8
-set -e -o pipefail
+set -o errexit -o pipefail
(
cd "$DISTSRC"
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.