On Tuesday 24 of April 2012, Tor Lillqvist wrote:
>
>     Hardcode cairo canvas off for Mac OS X and Windows
>
>     Do not depend on a distro-configs file to disable it. It is supposed
>     to be optional to pass a --with-distro option. Running a bare
>     ./autogen.sh is supposed to produce a working configuration.
...
> +if test $_os = Darwin -o $_os = WINNT; then
> +    if test "$enable_cairo_canvas" = yes; then
> +        AC_MSG_WARN([Overriding with -disable-cairo-canvas for this
> platform]) 

 Why is AC_MSG_ERROR not here? If somebody explicitly 
passed --enable-cairo-canvas, they most probably do not want it simply 
switched back. I mean, either the option is a mistake, in which case it 
shouldn't be passed anyway, or they passed it for a reason, in which case 
they should be told it's not making any effect.

> +    fi 
> +    enable_cairo_canvas=no
> +elif test -z "$enable_cairo_canvas"; then
> +    enable_cairo_canvas=yes


-- 
 Lubos Lunak
 l.lu...@suse.cz
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to