Thanks for the submission. I like i & iii and can input the changes (unless there is another volunteer). I have some misgivings about ii --adding a gl-specific customization for libs and includes. Do you object to CFLAGS, CPPFLAGS and LDFLAGS for this purpose (and which also accommodate many other customizations)? Regards, Pete
Jose Gomes wrote: > Hello, > > I have dx built, installed and running for linux and solaris > and would like to report 3 problems. The fixes can be found > in the file in attachment (which is the output of "cvs diff -C3"). > Please let me know if something is wrong with this. > I hope this will help. > > Best regards, > > Jose Gomes > > ----------------------------------------------------------------------- > Following are the three bugs description, labeled i), ii) and iii). > > i) > > The main problem is that the Makefile.am's are not written > to support both builddir==srcdir and builtdir!=srcdir > which was required for me since I wanted > to compile it for two architectures and which is (as far as I > know) the standard way to use the gnu tools. > So I had to change in almost every Makefile.am's > lines like: > INCLUDES = -I../../../include [EMAIL PROTECTED]@ > by lines like: > INCLUDES = -I$(srcdir)/../../../include [EMAIL PROTECTED]@ > Another modification is due to the fact that c++ sources are generated > automatically by a script called "class" which asssumed that > srcdir==builddir. > So this script is slightly modified so that it can > be passed the srcdir directory as argument. > See below for a list of the modified files. > > ii) > > My opengl headers and libraries are not is a standard place. > I added the options --with-gl-includes and --with-gl-libs > to the "configure" script so that these locations can be passed > to all Makefile.am through the variables GL_INCLUDES and GL_LIBS. > See bellow for a list of modified files. > > iii) > > I have flex installed for solaris and the scripts think that > linux implies flex and solaris implies lex. > The problem is that flex and lex are not compatible. > The file src/uipp/dxuilib/Network.C uses the "yylineno" > variable which does not exists in flex but does in lex. > This was taken into account in this C file by: > #if defined(linux) || defined(cygwin) || defined(freebsd) > int yylineno; > #else > extern int yylineno; /* lexer line number */ > #endif > I think it is not the correct test since the fact "yylineno" exists > is just a matter of "lex" or "flex". > So, instead of adding the test "|| defined(solaris)", I hadded the > autoconf variable "HAVE_YYLINENO" (initialized by "configure") > and replace the previous code by: > #ifndef HAVE_YYLINENO > int yylineno; > #else > extern int yylineno; /* lexer line number */ > #endif > See below for a list of modified files. > > Following are the list of modified files for fixing the 3 bugs. > > builddir==srcdir > --------------- > > M acconfig.h > M configure.in > M src/exec/dpexec/Makefile.am > M src/exec/dpexec/local.mk > M src/exec/dxexec/Makefile.am > M src/exec/dxmods/Makefile.am > M src/exec/hwrender/Makefile.am > M src/exec/hwrender/opengl/Makefile.am > M src/exec/libdx/Makefile.am > M src/exec/libdx/class > M src/exec/libdx/local.mk > M src/misc/Makefile.am > M src/uipp/base/Makefile.am > M src/uipp/dxl/Makefile.am > M src/uipp/dxui/Makefile.am > M src/uipp/dxuilib/Makefile.am > M src/uipp/dxuilib/Network.C > M src/uipp/java/Makefile.am > M src/uipp/mb/Makefile.am > > Non standard gl location. > ------------------------- > > M configure.in > M src/exec/dxexec/Makefile.am > M src/exec/hwrender/Makefile.am > M src/exec/hwrender/opengl/Makefile.am > > No yylineno in flex. > -------------------- > > M acconfig.h > M configure.in > M src/uipp/dxuilib/Network.C > > -- > [EMAIL PROTECTED] Tel. +33 4 92 38 76 48 > http://www.inria.fr/robotvis/personnel/jgomes > Projet RobotVis, INRIA, 2004, route des Lucioles - B.P. 93 > F-06902 Sophia Antipolis Cedex, FRANCE >