Ah. I see why it worked for me. I was using the MacPorts flex:
vygr:build hjelmn$ type -p flex
/opt/local/bin/flex
vygr:build hjelmn$ flex --version
flex 2.5.35

I will take a look tonight (if you haven't already fixed it) and see if I can 
improve find_and_check.

-Nathan
________________________________________
From: devel-boun...@open-mpi.org [devel-boun...@open-mpi.org] on behalf of 
Ralph Castain [r...@open-mpi.org]
Sent: Wednesday, November 14, 2012 4:26 PM
To: de...@open-mpi.org
Subject: Re: [OMPI devel] [OMPI svn] svn:open-mpi r27601 - trunk

Sorry Nathan - I had to revert this out as it broke builds on Mac ML. The 
problem is that the find_and_check parser looks for parens to find the version 
number, expecting something like this:

$ m4 --version
m4 (GNU M4) 1.4.16

or this:

$ gcc --version
i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1

However, on Mac ML, you get this for flex:

$ flex --version
flex 2.5.35 Apple(flex-31)

And so the parser incorrectly rejects the flex version. We'll have to come up 
with a more robust way of getting version numbers so we can do this test.



On Nov 12, 2012, at 11:28 PM, svn-commit-mai...@open-mpi.org wrote:

> Author: hjelmn (Nathan Hjelm)
> Date: 2012-11-13 02:28:10 EST (Tue, 13 Nov 2012)
> New Revision: 27601
> URL: https://svn.open-mpi.org/trac/ompi/changeset/27601
>
> Log:
> enforce minimum flex version in autogen.pl
>
> Text files modified:
>  trunk/autogen.pl |     4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
>
> Modified: trunk/autogen.pl
> ==============================================================================
> --- trunk/autogen.pl  Tue Nov 13 02:21:10 2012        (r27600)
> +++ trunk/autogen.pl  2012-11-13 02:28:10 EST (Tue, 13 Nov 2012)      (r27601)
> @@ -56,11 +56,13 @@
> my $ompi_automake_version = "1.11.1";
> my $ompi_autoconf_version = "2.65";
> my $ompi_libtool_version = "2.2.6b";
> +my $ompi_flex_version = "2.5.35";
>
> # Search paths
> my $ompi_autoconf_search = "autoconf";
> my $ompi_automake_search = "automake";
> my $ompi_libtoolize_search = "libtoolize;glibtoolize";
> +my $ompi_flex_search = "flex";
>
> # One-time setup
> my $username;
> @@ -797,6 +799,7 @@
>    GNU Autoconf: $ompi_autoconf_version
>    GNU Automake: $ompi_automake_version
>    GNU Libtool: $ompi_libtool_version
> +    Flex: $ompi_flex_version
> =================================================================\n";
>    my_exit(1);
> }
> @@ -1015,6 +1018,7 @@
> &find_and_check("autoconf", $ompi_autoconf_search, $ompi_autoconf_version);
> &find_and_check("libtool", $ompi_libtoolize_search, $ompi_libtool_version);
> &find_and_check("automake", $ompi_automake_search, $ompi_automake_version);
> +&find_and_check("flex", $ompi_flex_search, $ompi_flex_version);
>
> #---------------------------------------------------------------------------
>
> _______________________________________________
> svn mailing list
> s...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/svn


_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel

Reply via email to