[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 08/28] tests/tcg: Fix some targets default cross compiler
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v1 08/28] tests/tcg: Fix some targets default cross compiler path |
Date: |
Tue, 26 Oct 2021 12:41:13 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 |
On 10/26/21 12:22, Alex Bennée wrote:
> From: Philippe Mathieu-Daudé <f4bug@amsat.org>
>
> We do not want a shell command substitution, but a parameter
> substitution (with assignment). Replace $() -> ${}, otherwise
> the expanded command return an empty string and the $cross_cc
> variable is not set.
>
> Fixes: 634ef789f8e ("tests/tcg: add more default compilers to configure.sh")
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> Message-Id: <20211023164329.328137-1-f4bug@amsat.org>
> ---
> tests/tcg/configure.sh | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
- [PATCH v1 03/28] tests/docker: Update debian-hexagon-cross to a newer toolchain, (continued)
- [PATCH v1 03/28] tests/docker: Update debian-hexagon-cross to a newer toolchain, Alex Bennée, 2021/10/26
- [PATCH v1 12/28] ebpf: really include it only in system emulators, Alex Bennée, 2021/10/26
- [PATCH v1 09/28] tests/docker: split PARTIAL into PARTIAL and VIRTUAL images, Alex Bennée, 2021/10/26
- [PATCH v1 25/28] plugins: add helper functions for coverage plugins, Alex Bennée, 2021/10/26
- [PATCH v1 08/28] tests/tcg: Fix some targets default cross compiler path, Alex Bennée, 2021/10/26
- Re: [PATCH v1 08/28] tests/tcg: Fix some targets default cross compiler path,
Philippe Mathieu-Daudé <=
- [PATCH v1 17/28] docs/tcg-plugins: add L2 arguments to cache docs, Alex Bennée, 2021/10/26
- [PATCH v1 18/28] chardev: don't exit() straight away on C-a x, Alex Bennée, 2021/10/26
- [PATCH v1 19/28] tests/plugins: extend the insn plugin to track opcode sizes, Alex Bennée, 2021/10/26
- [PATCH v1 26/28] contrib/plugins: add a drcov plugin, Alex Bennée, 2021/10/26
- [PATCH v1 28/28] gdbstub: Switch to the thread receiving a signal, Alex Bennée, 2021/10/26
- [PATCH v1 15/28] plugins/cache: split command line arguments into name and value, Alex Bennée, 2021/10/26
- [PATCH v1 13/28] plugins/cache: freed heap-allocated mutexes, Alex Bennée, 2021/10/26
- [PATCH v1 23/28] tracing: remove the trace-tcg includes from the build, Alex Bennée, 2021/10/26