[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by th
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell |
Date: |
Thu, 20 Jul 2017 16:34:40 +0100 |
On 14 July 2017 at 11:45, Peter Maydell <address@hidden> wrote:
> In various places in our test makefiles and scripts we use the
> shell $RANDOM to create a random number. This is a bash
> specific extension, and doesn't work on other shells.
> With dash the shell doesn't complain, it just effectively
> always evaluates $RANDOM to 0:
> echo $((RANDOM + 32768)) => 32768
>
> However, on NetBSD the shell will complain:
> "-sh: arith: syntax error: "RANDOM + 32768"
>
> which means that "make check" fails.
>
> Switch to using "${RANDOM:-0}" instead of $RANDOM,
> which will portably either give us a random number or zero.
> This means that on non-bash shells we don't get such
> good test coverage via the MALLOC_PERTURB_ setting, but
> we were already in that situation for non-bash shells.
>
> Our only other uses of $RANDOM (in tests/qemu-iotests/check
> and tests/qemu-iotests/162) are in shell scripts which use
> a #!/bin/bash line so they are always run under bash.
>
> Suggested-by: Eric Blake <address@hidden>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> tests/Makefile.include | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Applied, thanks.
-- PMM
- [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell, Peter Maydell, 2017/07/14
- Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell, Kamil Rytarowski, 2017/07/14
- Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell, Fam Zheng, 2017/07/14
- Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell, Eric Blake, 2017/07/14
- Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell, Stefan Hajnoczi, 2017/07/17
- Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell, Markus Armbruster, 2017/07/20
- Re: [Qemu-devel] [PATCH] tests: Handle $RANDOM not being supported by the shell,
Peter Maydell <=