<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.32.2">
</HEAD>
<BODY>
On Sat, 2011-09-24 at 21:06 -0400, Matt Turner wrote:<BR>
<BLOCKQUOTE TYPE=CITE>
    <TT><FONT COLOR="#3c3c3c">The patch has a few problems currently, and a few things that can possibly be</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">done better:</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;- Mainly, that building libmesa.a currently fails.</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;- Not sure how to handle shared/static dricore options.</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;- libtool defines VERSION (-DVERSION=...), which screws up the VERSION</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; token in glsl_lexer.ll and glsl_parser.yy. I trivially renamed it.</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;- libralloc.la can probably be combined into libglslcore.la, and not</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; have to be added to every _LDADD line.</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;- The rules for flex and bison can probably be eliminated by using</FONT></TT><BR>
    <TT><FONT COLOR="#3c3c3c">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; YFLAGS and LFLAGS. I tried, but ylwrap gave me some error.</FONT></TT><BR>
    <BR>
</BLOCKQUOTE>
I had a quick look, configure.ac is huge and has a big impact on Makefiles. I think it should be reviewed and cleaned-up to understand how it affects makefiles.<BR>
<BR>
Just a few examples of statements to investigate. Not that they are bad, but they may be needed because there was no automake and they could possibly conflict.<BR>
<BLOCKQUOTE>
    AC_CHECK_PROGS([MAKE], [gmake make])<BR>
    <BR>
    AC_PATH_PROG([MKDEP], [makedepend])<BR>
    &nbsp;&nbsp;&nbsp; Can 'makedepend' impact the dependencies automake generates?<BR>
    <BR>
    AC_PATH_PROG([FLEX], [flex])<BR>
    AC_PATH_PROG([BISON], [bison])<BR>
    &nbsp;&nbsp;&nbsp; Some platforms have different names. The rules in the automake make file may not work if configure.ac does not use AC_PROG_YACC and AC_PROG_LEX<BR>
    This is the configuration in xorg:
    <BLOCKQUOTE>
<PRE>
AC_PROG_YACC
AC_PATH_PROG([YACC_INST], $YACC)
if test ! -f &quot;$srcdir/gram.c&quot;; then
&nbsp;&nbsp; if test -z &quot;$YACC_INST&quot;; then
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; AC_MSG_ERROR([yacc not found - unable to compile gram.y])
&nbsp;&nbsp; fi
fi
AC_PROG_LEX
</PRE>
    </BLOCKQUOTE>
    test &quot;x$INSTALL&quot; = &quot;x$ac_install_sh&quot; &amp;&amp; INSTALL='$(MINSTALL)'<BR>
    &nbsp;&nbsp;&nbsp; On some platforms, this may result in a different install program being used whether the makefile is from mesa or automake. <BR>
    <BR>
    There is a 'configuration' file. How will the automake makefiles take that into account.<BR>
</BLOCKQUOTE>
It looks like there are 'conflicts' between mesa makefiles and automake which is expected due to the size of the project. Some preparation work should be done upfront to make both system coexist and then have automake gradually replace mesa makefiles. This would be useful work anyway.<BR>
<BR>
<BR>
<BR>
<BR>
</BODY>
</HTML>