You're right, I think the general policy is to disable a language that
can't be configured, rather than fail. I've modified the patch to do just
that.


On 27 April 2014 22:30, Olly Betts <notifications@github.com> wrote:

> In configure.ac:
>
> > @@ -2041,7 +2044,10 @@ else
> >        GOGCC=true
> >        GOVERSIONOPTION=--version
> >        AC_MSG_CHECKING([whether gccgo version is too old])
> > -      go_version=`$GO $GOVERSIONOPTION | sed -e 's/[^0-9]* \([0-9.]*\) .*$/\1/' -e 's/[.]//g'`
> > +      go_version=[`$GO $GOVERSIONOPTION | sed -n '1p' | sed -e 's/^.* \([0-9.]*\) *$/\1/' -e 's/[.]//g'`]
> > +      if test "x$go_version" = x; then
> > +        AC_MSG_ERROR([could not determine gccgo version])
>
> Do we really want to fail configure in this case? The only existing cases
> when we use AC_MSG_ERROR are when a use supplied option to configure is
> invalid in some way. Otherwise we report the failure with AC_MSG_RESULTand just disable tests for the language in question.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/swig/swig/pull/166/files#r12031163>
> .
>