[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 19/23] tests/docker: invoke the DEBUG shell w
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v2 19/23] tests/docker: invoke the DEBUG shell with --noprofile/--norc |
Date: |
Tue, 23 Jul 2019 09:03:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 7/17/19 3:43 PM, Alex Bennée wrote:
> It's very confusing when things work in the debug shell because the
> environment is different from what the test is running. Fix this by
> ensuring we only have the inherited environment from the run shell.
>
> Signed-off-by: Alex Bennée <address@hidden>
> ---
> tests/docker/run | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/docker/run b/tests/docker/run
> index 7aebf4b5698..1014871fec0 100755
> --- a/tests/docker/run
> +++ b/tests/docker/run
> @@ -62,7 +62,7 @@ echo "* Prepared to run command:"
> echo " $CMD"
> echo "* Hit Ctrl-D to continue, or type 'exit 1' to abort"
> echo
> -$SHELL
> +$SHELL --noprofile --norc
>
> if "$CMD"; then
> exit 0
> @@ -72,7 +72,7 @@ elif test -n "$DEBUG"; then
> echo "* Hit Ctrl-D to exit"
> echo
> # Force error after shell exits
> - $SHELL && exit 1
> + $SHELL --noprofile --norc && exit 1
> else
> exit 1
> fi
>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
- [Qemu-devel] [PATCH v2 13/23] buildsys: The NSIS Windows build requires the documentation installed, (continued)
- [Qemu-devel] [PATCH v2 19/23] tests/docker: invoke the DEBUG shell with --noprofile/--norc, Alex Bennée, 2019/07/17
- Re: [Qemu-devel] [PATCH v2 19/23] tests/docker: invoke the DEBUG shell with --noprofile/--norc,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v2 21/23] tests/qemu-iotests/group: Remove some more tests from the "auto" group, Alex Bennée, 2019/07/17
- Re: [Qemu-devel] [PATCH v2 00/23] testing/next for 4.1-rc2 (win, travis, iotests), Alex Bennée, 2019/07/22