Stefan Manegold wrote:
On Fri, May 01, 2009 at 05:56:16AM +0200, Testweb Compilation wrote:
from lex.pf.c:19:/usr/include/features.h:150:1: error: this is the location of the previous definition make[5]: *** [libparser_la-scanner.yy.lo] Error 1
http://www.ins.cwi.nl/~monet/permastore/2009-05-01/Stable/make-pathfinder-GN...
CVS committers: fabian@cwi.nl manegold@cwi.nl
module maintainers: lsidir@cwi.nl zhang@cwi.nl henning@cwi.nl
The problem here is that in pathfinder, pf_config.h is in most C files (possibly at least all C files in pathfinder/compiler/ not included directly, but via pathfinder/compilier/include/pathfinder.h . Hence, our "trick" in buildtools/conf/rules.mk, to push the inclusion of pf_config.h (also) for flex, bison, swig, etc. -generated C files to the veru front of the file, does not work properly, i.e., pf_config.h get includes twice, once "forced" directly at the beginning of the (generated) C file, and once via pathfinder.h .
I see three solutions:
Remove '#include "pf_config.h"' from pathfinder/compilier/include/pathfinder.h and add '#include "pf_config.h"' as first include in all source files in pathfinder// (e.g., just before the existing '#include "pathfinder.h"'.
Make buildtools/conf/rules.mk, aware of pathfinder.h and handle is just like $(CONFIG_H)
(Try to) make bootstrap (i.e., auto*) a conf/config.h.in (or make configure create the <pkg>_config.h from conf/config.h.in) in such a way that it is resistent against multiple inclusion, e.g., using the usual #ifndef #define ... #endif mechanism.
I have a strong preference for solution one, as that is what we do the the remainder of the code base.
Any comments?
I concur.
Any volunteers?
I can do it.