[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/3] configure: fix forSunOS based systems
From: |
Thomas Huth |
Subject: |
Re: [PATCH 0/3] configure: fix forSunOS based systems |
Date: |
Mon, 27 Jul 2020 16:05:28 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
Hi David,
On 18/07/2020 15.19, David CARLIER wrote:
> From 63a3c9639d708a796abd958607aa6376fc9b99f2 Mon Sep 17 00:00:00 2001
> From: David Carlier <devnexen@gmail.com>
> Date: Sat, 18 Jul 2020 13:27:52 +0100
These headers --^ should not be part of the e-mail body (otherwise they
will be part of the patch description if this patch gets picked up with
"git am").
> Subject: [PATCH 1/3] configure: fix for SunOS based systems.
>
> local directive make the configure fails on these systems.
>
> Signed-off-by: David Carlier <devnexen@gmail.com>
> ---
> configure | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index e93836aaae..5d100eba72 100755
> --- a/configure
> +++ b/configure
> @@ -59,7 +59,7 @@ error_exit() {
> do_compiler() {
> # Run the compiler, capturing its output to the log. First argument
> # is compiler binary to execute.
> - local compiler="$1"
> + compiler="$1"
> shift
> if test -n "$BASH_VERSION"; then eval '
> echo >>config.log "
>
We're not using $compiler anywhere else outside of this function in the
configure script, so I think this should be ok.
Reviewed-by: Thomas Huth <thuth@redhat.com>