On Mon,  9 Jul 2012 19:24:47 +0200
Andreas Schneider <[email protected]> wrote:

> From: Andreas Schneider <[email protected]>
> 
> AC_PROG_SED is only avaliable in recent autoconf versions.
> Use AC_CHECK_PROG instead if AC_PROG_SED is not present.
> 
> Signed-off-by: Andreas Schneider <[email protected]>
> ---
>  configure.ac |    7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 0dd1155..f95a2de 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -44,10 +44,15 @@ AC_ARG_VAR(ROOTSBINDIR, [Location where files ordinarily 
> under /sbin should be i
>  
>  # Checks for programs.
>  AC_PROG_CC
> -AC_PROG_SED
>  AC_GNU_SOURCE
>  AM_PROG_CC_C_O
>  
> +# AC_PROG_SED is only avaliable in recent autoconf versions.
> +# Use AC_CHECK_PROG instead if AC_PROG_SED is not present.
> +ifdef([AC_PROG_SED],
> +      [AC_PROG_SED],
> +      [AC_CHECK_PROG(SED, sed, sed)])
> +
>  # Checks for typedefs, structures, and compiler characteristics.
>  AC_HEADER_STDBOOL
>  AC_TYPE_UID_T

Looks fine...

The AC_PROG_SED in the later versions of autoconf do quite a bit more
to verify that sed actually works, but our use of sed is very simple in
cifs-utils so this should be ok.

I'll plan to commit the patch in a day or two if no one objects.

Thanks,
-- 
Jeff Layton <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to