[PATCH:xts 1/6] configure.ac: Quote argument to m4_pattern_forbid.

Aaron Plattner aplattner at nvidia.com
Thu Jan 5 09:52:42 PST 2012


On 12/22/2011 04:23 PM, Cyril Brulebois wrote:
> Without this, configure spits out:
>    ./configure: line 12801: ac_fn_c_check_member: command not found
>    ./configure: line 12815: ac_fn_c_check_type: command not found

I'm confused.  It looks like the argument is already quoted and this change just 
adds the ^ and $ anchors.  I don't see these ac_fn_c_check_* errors unless I 
manually remove the [] quoting.

Did this commit message get copied from xserver commit 
8c51b886400c5962b31ff565771be1b01a3ca8fb ?

> Also anchor the pattern to make it stricter.

This seems fine as a standalone change.

> Signed-off-by: Cyril Brulebois<kibi at debian.org>
> ---
>   configure.ac |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index f3b7893..b2dbfc1 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -75,7 +75,7 @@ AC_DEFINE([XT_X_RELEASE], [6], [The X11 Release, e.g. 6 for X11R6])
>   AC_DEFINE([XTESTEXTENSION], [1], [Build tests using the XTEST extension])
>   AC_DEFINE([INPUTEXTENSION], [1], [Build tests using the XI extension])
>
> -m4_pattern_forbid([XTRANS_CONNECTION_FLAGS])
> +m4_pattern_forbid([^XTRANS_CONNECTION_FLAGS$])
>
>   # Transport selection macro from xtrans.m4
>   XTRANS_CONNECTION_FLAGS



More information about the xorg-devel mailing list