[PATCH wayland 3/3] configure: fix publican version detection

Pekka Paalanen ppaalanen at gmail.com
Sun Jul 6 03:32:44 PDT 2014


On Wed,  2 Jul 2014 17:23:46 +1000
Peter Hutterer <peter.hutterer at who-t.net> wrote:

> Publican now adds a spurious "v" to the version output.
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index c7ccace..e16c5b5 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -112,7 +112,7 @@ if test "x$enable_documentation" = "xyes"; then
>  	AC_PATH_PROG(PUBLICAN, publican)
>  
>  	if test "x$PUBLICAN" != "x"; then
> -		PUBLICAN_VERSION=[`$PUBLICAN -v | sed -e 's/version=\([0-9]*\.[0-9]*\).*/\1/'`]
> +		PUBLICAN_VERSION=[`$PUBLICAN -v | sed -e 's/version=v\?\([0-9]*\.[0-9]*\).*/\1/'`]
>  
>  		if test [ 1 -eq `echo "${PUBLICAN_VERSION} < 2.8" | bc` ]; then
>  			AC_MSG_ERROR([Publican version is not supported. Install publican >= 2.8 or disable the documentation using --disable-documentation])
> -- 
> 1.9.3

Hi,

looks good, though I can't test due to my lack of publican. All
three pushed.

To ssh://git.freedesktop.org/git/wayland/wayland
   113aac5..5679778  master -> master


Thanks,
pq


More information about the wayland-devel mailing list