Hi Cyril, Thanks for the changes. I've just started to look through them. One immediate concern I have about merging them is that it uses Standard C++ version of C headers, which from Standard C++ point is a good thing, but it can kick out support for compilers not quite supporting the Standard C++. Particular concerns are support for IRIX, Solaris, HP-UX, AIX, as I don't have direct access to any of these platforms I can't go test things out myself to provide a definitive answer what bits might work and what bits won't.
I'd suggest we raise this for a topic on the osg-users list and see what members of the community that have access to the full range of platforms say about what works/doesn't w.r.t using a Standard C++ versions of headers. Robert. On Dec 11, 2007 8:22 AM, Cyril Brulebois <[EMAIL PROTECTED]> wrote: > tag 454854 patch > thanks > > Hi Robert, > > first some context: > > On 08/12/2007, Martin Michlmayr wrote: > > Package: openscenegraph > > Version: 2.2.0-2 > > Usertags: ftbfs-gcc-4.3 > > > > Your package fails to build with GCC 4.3. Version 4.3 has not been > > released yet but I'm building with a snapshot in order to find errors > > and give people an advance warning. In GCC 4.3, the C++ header > > dependencies have been cleaned up. The advantage of this is that > > programs will compile faster. The downside is that you actually need > > to directly #include everything you use (but you really should > > do this anyway, otherwise your program won't work with any compiler > > other than GCC). There's some more information about this at > > http://www.cyrius.com/journal/2007/05/10#gcc-4.3-include > > > > You can reproduce this problem with gcc-snapshot from unstable. Note > > that Red Hat, Novell and Ubuntu have done some work getting packages > > to build with GCC 4.3 so there might be patches floating around > > somewhere. I suggest you talk to your upstream. > > Please find attached two patches to make osg 2.2 build with gcc 4.3. > Mostly some includes are needed. Some functions can't be static as I > understand it. And there's an extra typedef which doesn't seem to make > the build fail, but triggers many warnings while building, which is a > bit annoying; I've put this one in a separate patch for this reason. > > Cheers, > > -- > Cyril Brulebois > -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
