[Mesa-dev] [PATCH] configure.ac: use AC_PROG_SED

Kenneth Graunke kenneth at whitecape.org
Tue Jan 10 22:55:11 PST 2012


On 01/10/2012 10:16 PM, Matt Turner wrote:
> AC_PROG_PATH was used in the original configure.ac (dca1b796b) and I
> can't see any reason AC_PROG_SED wasn't used.

It looks like AC_PROG_SED was added in 2.59b, and wasn't in the original 
2.59.  Presumably that's why, though it could've been an oversight.

At any rate, I see no reason not to use it now.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

> Signed-off-by: Matt Turner<mattst88 at gmail.com>
> ---
>   configure.ac |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index c0abeb9..08a0209 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -38,8 +38,8 @@ AC_PROG_CC
>   AC_PROG_CXX
>   AC_CHECK_PROGS([MAKE], [gmake make])
>   AC_CHECK_PROGS([PYTHON2], [python2 python])
> +AC_PROG_SED
>   AC_PATH_PROG([MKDEP], [makedepend])
> -AC_PATH_PROG([SED], [sed])
>
>   if test "x$MKDEP" = "x"; then
>       AC_MSG_ERROR([makedepend is required to build Mesa])


More information about the mesa-dev mailing list