diff mbox series

[19/24] configure: remove obsolete Solaris ar check

Message ID 20211007130829.632254-14-pbonzini@redhat.com (mailing list archive)
State New, archived
Headers show
Series configure->meson queue for 6.2 | expand

Commit Message

Paolo Bonzini Oct. 7, 2021, 1:08 p.m. UTC
Meson already has its own logic to find the "ar" binary, so remove the
Solaris specific check.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 configure | 15 ---------------
 1 file changed, 15 deletions(-)

Comments

Thomas Huth Oct. 7, 2021, 6:19 p.m. UTC | #1
On 07/10/2021 15.08, Paolo Bonzini wrote:
> Meson already has its own logic to find the "ar" binary, so remove the
> Solaris specific check.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   configure | 15 ---------------
>   1 file changed, 15 deletions(-)
> 
> diff --git a/configure b/configure
> index 016814d23b..2091844ad6 100755
> --- a/configure
> +++ b/configure
> @@ -2284,21 +2284,6 @@ EOF
>     fi
>   fi
>   
> -#########################################
> -# Solaris specific configure tool chain decisions
> -
> -if test "$solaris" = "yes" ; then
> -  if has ar; then
> -    :
> -  else
> -    if test -f /usr/ccs/bin/ar ; then
> -      error_exit "No path includes ar" \
> -          "Add /usr/ccs/bin to your path and rerun configure"
> -    fi
> -    error_exit "No path includes ar"
> -  fi
> -fi

Reviewed-by: Thomas Huth <thuth@redhat.com>
Marc-André Lureau Oct. 7, 2021, 8:41 p.m. UTC | #2
On Thu, Oct 7, 2021 at 5:17 PM Paolo Bonzini <pbonzini@redhat.com> wrote:

> Meson already has its own logic to find the "ar" binary, so remove the
> Solaris specific check.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>


> ---
>  configure | 15 ---------------
>  1 file changed, 15 deletions(-)
>
> diff --git a/configure b/configure
> index 016814d23b..2091844ad6 100755
> --- a/configure
> +++ b/configure
> @@ -2284,21 +2284,6 @@ EOF
>    fi
>  fi
>
> -#########################################
> -# Solaris specific configure tool chain decisions
> -
> -if test "$solaris" = "yes" ; then
> -  if has ar; then
> -    :
> -  else
> -    if test -f /usr/ccs/bin/ar ; then
> -      error_exit "No path includes ar" \
> -          "Add /usr/ccs/bin to your path and rerun configure"
> -    fi
> -    error_exit "No path includes ar"
> -  fi
> -fi
> -
>  if test "$tcg" = "enabled"; then
>      git_submodules="$git_submodules tests/fp/berkeley-testfloat-3"
>      git_submodules="$git_submodules tests/fp/berkeley-softfloat-3"
> --
> 2.31.1
>
>
>
>
diff mbox series

Patch

diff --git a/configure b/configure
index 016814d23b..2091844ad6 100755
--- a/configure
+++ b/configure
@@ -2284,21 +2284,6 @@  EOF
   fi
 fi
 
-#########################################
-# Solaris specific configure tool chain decisions
-
-if test "$solaris" = "yes" ; then
-  if has ar; then
-    :
-  else
-    if test -f /usr/ccs/bin/ar ; then
-      error_exit "No path includes ar" \
-          "Add /usr/ccs/bin to your path and rerun configure"
-    fi
-    error_exit "No path includes ar"
-  fi
-fi
-
 if test "$tcg" = "enabled"; then
     git_submodules="$git_submodules tests/fp/berkeley-testfloat-3"
     git_submodules="$git_submodules tests/fp/berkeley-softfloat-3"