[PATCH] scripts: Do not used set in sourced script

2 views
Skip to first unread message

Anton Mikanovich

unread,
Jun 10, 2024, 6:03:06 AMJun 10
to isar-...@googlegroups.com, Anton Mikanovich
Adding 'set -e' in the sourcing script makes it apply on running
console/chroot. This results in closing current console/chroot in case
of Isar build failure.
Partly revert 4f9b6419 removing this case for isar-buildenv-internal.

Signed-off-by: Anton Mikanovich <ami...@ilbers.de>
---
scripts/isar-buildenv-internal | 2 --
1 file changed, 2 deletions(-)

diff --git a/scripts/isar-buildenv-internal b/scripts/isar-buildenv-internal
index 817387cf..bf8c3387 100755
--- a/scripts/isar-buildenv-internal
+++ b/scripts/isar-buildenv-internal
@@ -21,8 +21,6 @@

# It is assumed ISARROOT is already defined when this is called

-set -e
-
if [ -z "$ISARROOT" ]; then
echo >&2 "Error: ISARROOT is not defined!"
return 1
--
2.34.1

Uladzimir Bely

unread,
Jun 17, 2024, 1:24:31 AMJun 17
to Anton Mikanovich, isar-...@googlegroups.com
Applied to next.

--
Best regards,
Uladzimir.
Reply all
Reply to author
Forward
0 new messages