Re: [Fresco-devel] small patch for iso c++ header
Stefan Seefeld <[email protected]>
| Newsgroups | gmane.comp.video.fresco.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi Warren, Warren Turkal wrote: > This patch fixes a compile time warning from g++ 3.2 about using non-standard > c++ headers. thanks for the patch. The problem is unfortunately a bit more involved than that: Applying your patch would mean to lose the ability to compile Fresco with earlier versions of gcc. For that reason we didn't change the code yet, but rather use 'configure CXX="g++ -Wno-deprecated"' to suppress the warning. At some point we'll want to do what you suggest, though it's not yet decided when exactly that will be. > How does one get cvs write access. I would like to clean all of these > non-standard header warnings that I am getting. For simple changes like these, you may just send in patches (either to this list, or to our issue tracker at http://issues.fresco.org). CVS write access is usually granted to long term contributors who get seriously involved into the development. > Oh, also I get the following error during compile time when building if I > disable SDL in the configure: > make[1]: Leaving directory `/home/wt/src/fresco/Fresco/BUILD_DIR/GGI' > make: *** No rule to make target `SDL/Makefile', needed by `SDL'. Stop. Yeah, we should point out this issue in the INSTALL file: When you run 'configure --disable-SDL', the 'SDL' subdirectory shouldn't be created, so 'make' will not attempt to build it. You probably configured without that parameter, and so configure bailed out in an attempt to configure the SDL subpackage, which only happens after the 'SDL/' subdirectory has been created. Unfortunately autoconf doesn't allow to automatically detect whether a subproject can be built, i.e. you have to explicitely disable it via '--disable-<package>'. Just remove SDL/ and run 'make' again, that should do the trick. Hope this helps, Stefan