[Mesa-dev] [PATCH] configure.ac: Remove space after indent -T flag.

Matt Turner mattst88 at gmail.com
Mon Dec 31 17:31:02 PST 2012


On Mon, Dec 31, 2012 at 3:27 PM, Vinson Lee <vlee at freedesktop.org> wrote:
> Fixes this build error on platforms not using GNU indent.
>
> indent: Command line: ``-T'' requires a parameter
>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index 1294740..87bb71f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -73,7 +73,7 @@ AC_PATH_PROG([PERL], [perl])
>
>  AC_CHECK_PROG(INDENT, indent, indent, cat)
>  if test "x$INDENT" != "xcat"; then
> -    AC_SUBST(INDENT_FLAGS, '-i4 -nut -br -brs -npcs -ce -T GLubyte -T GLbyte -T Bool')
> +    AC_SUBST(INDENT_FLAGS, '-i4 -nut -br -brs -npcs -ce -TGLubyte -TGLbyte -TBool')
>  fi
>
>  dnl Our fallback install-sh is a symlink to minstall. Use the existing
> --
> 1.8.0

Fine by me. I've seen a report that -brs is a GNU-only option and
doesn't work with BSD indent. What indent are you using that doesn't
accept the space after -T but does accept -brs?

https://bugs.gentoo.org/show_bug.cgi?id=428112


More information about the mesa-dev mailing list