https://bugs.freedesktop.org/show_bug.cgi?id=36651
--- Comment #3 from Dan Nicholson <dbn.li...@gmail.com> 2011-05-09 05:56:58 PDT --- Review of attachment 46450: --> (https://bugs.freedesktop.org/review?bug=36651&attachment=46450) In order to make the substitution work, you need to add a place for configure to put the definition of found programs. In configs/autoconf, add these two lines somewhere: FLEX = @FLEX@ BISON = @BISON@ The rest looks good to me. There are ways to make this more portable with autoconf, but we can save that for another patch. ::: configure.ac @@ +34,3 @@ AC_CHECK_PROGS([PYTHON2], [python2 python]) +AC_CHECK_PROGS([FLEX], [flex]) +AC_CHECK_PROGS([BISON], [bison]) There are some autoconf macros specifically for lex/yacc, but this should do for now. However, what's missing here is that we're not checking if the tools are not available and stopping if they're not available. configure will set the program to "no" if it finds nothing. Also, I'm not sure it's a big deal, but I prefer AC_PATH_PROG if we're not checking for multiple program names. Something like this might work: AC_PATH_PROG([FLEX], [flex]) test "x$FLEX" = "xno" && AC_MSG_ERROR([flex is needed to build mesa]) AC_PATH_PROG([BISON], [bison]) test "x$BISON" = "xno" && AC_MSG_ERROR([bison is needed to build mesa]) -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
