Re: [PATCH v1 13/18] website-build: configure: make realpath and tar requirement work.

[Adrien 'neox' Bourmault]

Le samedi 25 mai 2024 à 20:25 +0200, Denis 'GNUtoo' Carikli a écrit :
> Before this change, we wanted to require realpath and tar and we check
> for both but it didn't fail if they are not found.
> 
> Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
> ---
>  website-build/configure.ac | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/website-build/configure.ac b/website-build/configure.ac
> index 3036ee5..43d3300 100644
> --- a/website-build/configure.ac
> +++ b/website-build/configure.ac
> @@ -68,8 +68,13 @@ AC_ARG_WITH([untitled-path],
>  AM_CONDITIONAL( [WANT_UNTITLED_PATH], [test x"$UNTITLED_PATH" != x""])
>  
>  # Check dependencies
> -AC_CHECK_PROG([REALPATH], [realpath], [realpath])
> -AC_CHECK_PROG([TAR], [tar], [tar])
> +AC_CHECK_PROG([FOUND_REALPATH], [realpath], [realpath])
> +AS_IF([test x"$FOUND_REALPATH" = x""],
> +      [AC_MSG_ERROR([realpath was not found in PATH ($PATH)])])
> +
> +AC_CHECK_PROG([FOUND_TAR], [tar], [tar])
> +AS_IF([test x"$FOUND_TAR" = x""],
> +      [AC_MSG_ERROR([tar was not found in PATH ($PATH)])])
>  
>  AS_IF([test x"$guix" = x"yes"],
>        [AC_CHECK_PROG([FOUND_GUIX], [guix], [guix])
> @@ -147,11 +152,6 @@ AS_IF([test x"$guix" = x"yes"],
>               [AC_MSG_ERROR(
>                [rm was not found in PATH ($PATH)])])
>  
> -       AC_CHECK_PROG([FOUND_SED], [sed], [sed])
> -       AS_IF([test x"$FOUND_SED" = x""],
> -             [AC_MSG_ERROR(
> -              [sed was not found in PATH ($PATH)])])
> -
>         AC_CHECK_PROG([FOUND_TAIL], [tail], [tail])
>         AS_IF([test x"$FOUND_TAIL" = x""],
>               [AC_MSG_ERROR(

Acked-by: Adrien 'neox' Bourmault <neox@gnu.org>

signature.asc
Description: This is a digitally signed message part