On Fri, 19 Sep 2025 15:45:04 -0600
Leo Sandoval via Grub-devel <[email protected]> wrote:
> In this case, it does not hurt to increase bash execution verbosity so
> we can get more insight in case of issues.
I also don't think this is a good change either, and would prefer it be
reverted. But I'm less opinionated on it compared to the other two,
which should certainly be reverted.
I also have had the problem that this is trying to solve. I've written
a set of scripts to run the GRUB tests that allow for enabling shell
tracing, without needing to make this change. Instead of this change,
I'd be more in favor of one that allows conditional enabling of
tracing, with tracing off by default. This could be as simple as
changing every test file to have something like `[ "x${SHELL_TRACE}" =
"xy" ] && set -x`. I'd prefer an even less intrusive change, like
adding that condition in a log compiler (which is what my scripts
do). As it is, this change feels a bit like a sledge hammer.
Glenn
> Signed-off-by: Leo Sandoval <[email protected]>
> ---
> tests/ahci_test.in | 2 +-
> tests/asn1_test.in | 2 +-
> tests/btrfs_test.in | 2 +-
> tests/cdboot_test.in | 2 +-
> tests/core_compress_test.in | 2 +-
> tests/cpio_test.in | 2 +-
> tests/ehci_test.in | 2 +-
> tests/erofs_test.in | 2 +-
> tests/example_scripted_test.in | 2 +-
> tests/exfat_test.in | 2 +-
> tests/ext234_test.in | 2 +-
> tests/f2fs_test.in | 2 +-
> tests/fat_test.in | 2 +-
> tests/fddboot_test.in | 2 +-
> tests/file_filter_test.in | 2 +-
> tests/grub_cmd_date.in | 2 +-
> tests/grub_cmd_regexp.in | 2 +-
> tests/grub_cmd_set_date.in | 2 +-
> tests/grub_cmd_sleep.in | 2 +-
> tests/grub_cmd_test.in | 2 +-
> tests/grub_func_test.in | 2 +-
> tests/grub_script_blanklines.in | 2 +-
> tests/grub_script_blockarg.in | 2 +-
> tests/grub_script_dollar.in | 2 +-
> tests/grub_script_expansion.in | 2 +-
> tests/grub_script_final_semicolon.in | 2 +-
> tests/grub_script_no_commands.in | 2 +-
> tests/gzcompress_test.in | 2 +-
> tests/hddboot_test.in | 2 +-
> tests/help_test.in | 2 +-
> tests/hfs_test.in | 2 +-
> tests/hfsplus_test.in | 2 +-
> tests/iso9660_test.in | 2 +-
> tests/jfs_test.in | 2 +-
> tests/luks1_test.in | 2 +-
> tests/luks2_test.in | 2 +-
> tests/lzocompress_test.in | 2 +-
> tests/minixfs_test.in | 2 +-
> tests/netboot_test.in | 2 +-
> tests/nilfs2_test.in | 2 +-
> tests/ntfs_test.in | 2 +-
> tests/ohci_test.in | 2 +-
> tests/partmap_test.in | 2 +-
> tests/pata_test.in | 2 +-
> tests/pseries_test.in | 2 +-
> tests/reiserfs_test.in | 2 +-
> tests/romfs_test.in | 2 +-
> tests/serial_test.in | 2 +-
> tests/squashfs_test.in | 2 +-
> tests/syslinux_test.in | 2 +-
> tests/tar_test.in | 2 +-
> tests/test_sha512sum.in | 2 +-
> tests/udf_test.in | 2 +-
> tests/uhci_test.in | 2 +-
> tests/xfs_test.in | 2 +-
> tests/xzcompress_test.in | 2 +-
> tests/zfs_test.in | 2 +-
> 57 files changed, 57 insertions(+), 57 deletions(-)
>
> diff --git a/tests/ahci_test.in b/tests/ahci_test.in
> index 70646a24ec..ffb7cf48d6 100644
> --- a/tests/ahci_test.in
> +++ b/tests/ahci_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/asn1_test.in b/tests/asn1_test.in
> index 8f18ee6bb6..a9b82dd0e4 100644
> --- a/tests/asn1_test.in
> +++ b/tests/asn1_test.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> . "@builddir@/grub-core/modinfo.sh"
>
> diff --git a/tests/btrfs_test.in b/tests/btrfs_test.in
> index a07d2e5d18..84cfd95d8e 100644
> --- a/tests/btrfs_test.in
> +++ b/tests/btrfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/cdboot_test.in b/tests/cdboot_test.in
> index f00cdec587..ec8943b16f 100644
> --- a/tests/cdboot_test.in
> +++ b/tests/cdboot_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/core_compress_test.in b/tests/core_compress_test.in
> index 24a811418a..0b97c9557c 100644
> --- a/tests/core_compress_test.in
> +++ b/tests/core_compress_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/cpio_test.in b/tests/cpio_test.in
> index 5742cf17b9..fb468564a9 100644
> --- a/tests/cpio_test.in
> +++ b/tests/cpio_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if ! which cpio >/dev/null 2>&1; then
> echo "cpio not installed; cannot test cpio."
> diff --git a/tests/ehci_test.in b/tests/ehci_test.in
> index bf823a5de7..7aff6df941 100644
> --- a/tests/ehci_test.in
> +++ b/tests/ehci_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/erofs_test.in b/tests/erofs_test.in
> index 436468dac6..e3f3ea5eb4 100644
> --- a/tests/erofs_test.in
> +++ b/tests/erofs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/example_scripted_test.in b/tests/example_scripted_test.in
> index 783b7f1385..0f32c79fad 100644
> --- a/tests/example_scripted_test.in
> +++ b/tests/example_scripted_test.in
> @@ -1,4 +1,4 @@
> #!@BUILD_SHEBANG@
> -set -e
> +set -ex
>
> true
> diff --git a/tests/exfat_test.in b/tests/exfat_test.in
> index 7acde1977e..ae2b209a1e 100644
> --- a/tests/exfat_test.in
> +++ b/tests/exfat_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/ext234_test.in b/tests/ext234_test.in
> index c9a8c1056a..2f8aafc9c6 100644
> --- a/tests/ext234_test.in
> +++ b/tests/ext234_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/f2fs_test.in b/tests/f2fs_test.in
> index a020a0f96d..4d16c2fabe 100644
> --- a/tests/f2fs_test.in
> +++ b/tests/f2fs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/fat_test.in b/tests/fat_test.in
> index 77e3f16c67..0e4b1ebaed 100644
> --- a/tests/fat_test.in
> +++ b/tests/fat_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/fddboot_test.in b/tests/fddboot_test.in
> index 6ef49efcb2..74d2bcd874 100644
> --- a/tests/fddboot_test.in
> +++ b/tests/fddboot_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/file_filter_test.in b/tests/file_filter_test.in
> index ed6abcb5af..c2748621f6 100644
> --- a/tests/file_filter_test.in
> +++ b/tests/file_filter_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/grub_cmd_date.in b/tests/grub_cmd_date.in
> index 4903ad6cc3..359c624fce 100644
> --- a/tests/grub_cmd_date.in
> +++ b/tests/grub_cmd_date.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> . "@builddir@/grub-core/modinfo.sh"
>
> diff --git a/tests/grub_cmd_regexp.in b/tests/grub_cmd_regexp.in
> index 6520bd6d79..13633bb052 100644
> --- a/tests/grub_cmd_regexp.in
> +++ b/tests/grub_cmd_regexp.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # Run GRUB script in a Qemu instance
> # Copyright (C) 2010 Free Software Foundation, Inc.
> diff --git a/tests/grub_cmd_set_date.in b/tests/grub_cmd_set_date.in
> index 17673cd8aa..5572535fa4 100644
> --- a/tests/grub_cmd_set_date.in
> +++ b/tests/grub_cmd_set_date.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> . "@builddir@/grub-core/modinfo.sh"
>
> diff --git a/tests/grub_cmd_sleep.in b/tests/grub_cmd_sleep.in
> index 1a57fb388c..2da9d722ec 100644
> --- a/tests/grub_cmd_sleep.in
> +++ b/tests/grub_cmd_sleep.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> . "@builddir@/grub-core/modinfo.sh"
>
> diff --git a/tests/grub_cmd_test.in b/tests/grub_cmd_test.in
> index 043c3a634b..a78a5d6ee3 100644
> --- a/tests/grub_cmd_test.in
> +++ b/tests/grub_cmd_test.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # create a randome file
> empty="`mktemp "${TMPDIR:-/tmp}/tmp.XXXXXXXXXX"`" || exit 99
> diff --git a/tests/grub_func_test.in b/tests/grub_func_test.in
> index 1fa3c43527..6065ff6df1 100644
> --- a/tests/grub_func_test.in
> +++ b/tests/grub_func_test.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> . "@builddir@/grub-core/modinfo.sh"
>
> diff --git a/tests/grub_script_blanklines.in b/tests/grub_script_blanklines.in
> index bd8735491b..9b84782c38 100644
> --- a/tests/grub_script_blanklines.in
> +++ b/tests/grub_script_blanklines.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> @builddir@/grub-script-check <<EOF
> # comment 1
> diff --git a/tests/grub_script_blockarg.in b/tests/grub_script_blockarg.in
> index a91c7ca82a..1cc7c1a327 100644
> --- a/tests/grub_script_blockarg.in
> +++ b/tests/grub_script_blockarg.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # Run GRUB script in a Qemu instance
> # Copyright (C) 2010 Free Software Foundation, Inc.
> diff --git a/tests/grub_script_dollar.in b/tests/grub_script_dollar.in
> index 392fe2e7ab..fd8e5d8a42 100644
> --- a/tests/grub_script_dollar.in
> +++ b/tests/grub_script_dollar.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> @builddir@/grub-script-check << EOF
> echo "\\\$"
> diff --git a/tests/grub_script_expansion.in b/tests/grub_script_expansion.in
> index 98d5a9068a..452f8334ae 100644
> --- a/tests/grub_script_expansion.in
> +++ b/tests/grub_script_expansion.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # Run GRUB script in a Qemu instance
> # Copyright (C) 2010 Free Software Foundation, Inc.
> diff --git a/tests/grub_script_final_semicolon.in
> b/tests/grub_script_final_semicolon.in
> index f17a9bf95e..6e15b95de6 100644
> --- a/tests/grub_script_final_semicolon.in
> +++ b/tests/grub_script_final_semicolon.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> @builddir@/grub-script-check <<EOF
> echo one;
> diff --git a/tests/grub_script_no_commands.in
> b/tests/grub_script_no_commands.in
> index 996bb18104..b401f8feb4 100644
> --- a/tests/grub_script_no_commands.in
> +++ b/tests/grub_script_no_commands.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # grub-script-check refuses to pass a file with no commands; this usually
> # indicates a bug in the code generating that file.
> diff --git a/tests/gzcompress_test.in b/tests/gzcompress_test.in
> index d7a594bb2c..471b6537ed 100644
> --- a/tests/gzcompress_test.in
> +++ b/tests/gzcompress_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/hddboot_test.in b/tests/hddboot_test.in
> index 764e0da1c1..76704c379f 100644
> --- a/tests/hddboot_test.in
> +++ b/tests/hddboot_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/help_test.in b/tests/help_test.in
> index 9c8ca52c86..ca3d7f31b9 100644
> --- a/tests/help_test.in
> +++ b/tests/help_test.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> . "@builddir@/grub-core/modinfo.sh"
>
> diff --git a/tests/hfs_test.in b/tests/hfs_test.in
> index c916b9bac9..fc0e0e75c6 100644
> --- a/tests/hfs_test.in
> +++ b/tests/hfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/hfsplus_test.in b/tests/hfsplus_test.in
> index cb36a3661c..7918a68bbd 100644
> --- a/tests/hfsplus_test.in
> +++ b/tests/hfsplus_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/iso9660_test.in b/tests/iso9660_test.in
> index 793664c874..7a6816f94a 100644
> --- a/tests/iso9660_test.in
> +++ b/tests/iso9660_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if ! which xorriso >/dev/null 2>&1; then
> echo "xorriso not installed; cannot test iso9660."
> diff --git a/tests/jfs_test.in b/tests/jfs_test.in
> index 86f9ebeaa0..b1c18e9b16 100644
> --- a/tests/jfs_test.in
> +++ b/tests/jfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/luks1_test.in b/tests/luks1_test.in
> index 0bfb53c98c..3344746c10 100644
> --- a/tests/luks1_test.in
> +++ b/tests/luks1_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/luks2_test.in b/tests/luks2_test.in
> index 49db47380e..5368698f82 100644
> --- a/tests/luks2_test.in
> +++ b/tests/luks2_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/lzocompress_test.in b/tests/lzocompress_test.in
> index 42e270df02..50145dd0cd 100644
> --- a/tests/lzocompress_test.in
> +++ b/tests/lzocompress_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/minixfs_test.in b/tests/minixfs_test.in
> index 78577fec8c..0d28a72261 100644
> --- a/tests/minixfs_test.in
> +++ b/tests/minixfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/netboot_test.in b/tests/netboot_test.in
> index 510c9c34ba..b8a5f9abb8 100644
> --- a/tests/netboot_test.in
> +++ b/tests/netboot_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/nilfs2_test.in b/tests/nilfs2_test.in
> index 719972f53f..84b8b1a0b6 100644
> --- a/tests/nilfs2_test.in
> +++ b/tests/nilfs2_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/ntfs_test.in b/tests/ntfs_test.in
> index da73c59f9e..5b1bebe16c 100644
> --- a/tests/ntfs_test.in
> +++ b/tests/ntfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/ohci_test.in b/tests/ohci_test.in
> index a40d3bc0a0..1652b7efc2 100644
> --- a/tests/ohci_test.in
> +++ b/tests/ohci_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/partmap_test.in b/tests/partmap_test.in
> index 9810cc1acc..8e12cf7ec4 100644
> --- a/tests/partmap_test.in
> +++ b/tests/partmap_test.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # Copyright (C) 2010 Free Software Foundation, Inc.
> #
> diff --git a/tests/pata_test.in b/tests/pata_test.in
> index 4d0e7d5731..6c2fa43ddb 100644
> --- a/tests/pata_test.in
> +++ b/tests/pata_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/pseries_test.in b/tests/pseries_test.in
> index 9b4090cf50..a1dd1670a0 100644
> --- a/tests/pseries_test.in
> +++ b/tests/pseries_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/reiserfs_test.in b/tests/reiserfs_test.in
> index 36e34c3050..48d3ab4cc6 100644
> --- a/tests/reiserfs_test.in
> +++ b/tests/reiserfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/romfs_test.in b/tests/romfs_test.in
> index 98bb50c324..2c66cde345 100644
> --- a/tests/romfs_test.in
> +++ b/tests/romfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if ! which genromfs >/dev/null 2>&1; then
> echo "genromfs not installed; cannot test romfs."
> diff --git a/tests/serial_test.in b/tests/serial_test.in
> index 48655d4b97..331c3c3ad8 100644
> --- a/tests/serial_test.in
> +++ b/tests/serial_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/squashfs_test.in b/tests/squashfs_test.in
> index 2f044f95d9..b973f7bac0 100644
> --- a/tests/squashfs_test.in
> +++ b/tests/squashfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if ! which mksquashfs >/dev/null 2>&1; then
> echo "mksquashfs not installed; cannot test squashfs."
> diff --git a/tests/syslinux_test.in b/tests/syslinux_test.in
> index 44d3cdf7a5..e81c153107 100644
> --- a/tests/syslinux_test.in
> +++ b/tests/syslinux_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> outfile="`mktemp "${TMPDIR:-/tmp}/tmp.XXXXXXXXXX"`" || exit 99
>
> diff --git a/tests/tar_test.in b/tests/tar_test.in
> index 6e2f2de8b7..eafb6649dd 100644
> --- a/tests/tar_test.in
> +++ b/tests/tar_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if ! which tar >/dev/null 2>&1; then
> echo "tar not installed; cannot test tar."
> diff --git a/tests/test_sha512sum.in b/tests/test_sha512sum.in
> index b2bd89609c..3866f40c8d 100644
> --- a/tests/test_sha512sum.in
> +++ b/tests/test_sha512sum.in
> @@ -1,5 +1,5 @@
> #! @BUILD_SHEBANG@
> -set -e
> +set -ex
>
> # create a randome file
> file="`mktemp "${TMPDIR:-/tmp}/tmp.XXXXXXXXXX"`" || exit 99
> diff --git a/tests/udf_test.in b/tests/udf_test.in
> index 8968fb1033..b84d0b6746 100644
> --- a/tests/udf_test.in
> +++ b/tests/udf_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/uhci_test.in b/tests/uhci_test.in
> index de199a2810..46de98d921 100644
> --- a/tests/uhci_test.in
> +++ b/tests/uhci_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/xfs_test.in b/tests/xfs_test.in
> index 8a648aa4a7..d6b3ab908a 100644
> --- a/tests/xfs_test.in
> +++ b/tests/xfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
> diff --git a/tests/xzcompress_test.in b/tests/xzcompress_test.in
> index cfc6ccba66..f0ac52e2d0 100644
> --- a/tests/xzcompress_test.in
> +++ b/tests/xzcompress_test.in
> @@ -14,7 +14,7 @@
> # You should have received a copy of the GNU General Public License
> # along with GRUB. If not, see <http://www.gnu.org/licenses/>.
>
> -set -e
> +set -ex
> grubshell=@builddir@/grub-shell
>
> . "@builddir@/grub-core/modinfo.sh"
> diff --git a/tests/zfs_test.in b/tests/zfs_test.in
> index c8bb7937fe..e1cb766a5d 100644
> --- a/tests/zfs_test.in
> +++ b/tests/zfs_test.in
> @@ -1,6 +1,6 @@
> #!@BUILD_SHEBANG@
>
> -set -e
> +set -ex
>
> if [ "x$EUID" = "x" ] ; then
> EUID=`id -u`
_______________________________________________
Grub-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/grub-devel
