Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#179 shapes: Build error against Mesa >= 51b0a0b

closed-fixed
nobody
critical (31)
5
2012-11-16
2012-07-22
No

http://cgit.freedesktop.org/mesa/mesa/commit/?id=51b0a0b33ca16599a5972c3e77607284a1b7fb9b

[ 97s] Scanning dependencies of target shapes
[ 97s] [ 89%] Building C object CMakeFiles/shapes.dir/progs/demos/shapes/shapes.c.o
[ 97s] /home/abuild/rpmbuild/BUILD/freeglut/progs/demos/shapes/shapes.c:98:25: error: conflicting types for 'PFNGLSHADERSOURCEPROC'
[ 97s] In file included from /usr/include/GL/gl.h:2085:0,
[ 97s] from /home/abuild/rpmbuild/BUILD/freeglut/include/GL/freeglut_std.h:142,
[ 97s] from /home/abuild/rpmbuild/BUILD/freeglut/include/GL/freeglut.h:17,
[ 97s] from /home/abuild/rpmbuild/BUILD/freeglut/progs/demos/shapes/shapes.c:40:
[ 97s] /usr/include/GL/glext.h:6053:25: note: previous declaration of 'PFNGLSHADERSOURCEPROC' was here
[ 97s] make[2]: *** [CMakeFiles/shapes.dir/progs/demos/shapes/shapes.c.o] Error 1

Discussion

  • Thanks for the report. I assume this is on (somewhat) current trunk? Does the MESA library come with some preprocessor definitions that allow detecting it so that we can skip compiling this part of the demo code (getting the function adresses is really only necessary on windows)?

     
  • Dear Johannes,

    A fix has been applied to current trunk (rev 1385). Could you test it?

    Thanks!
    Dee

     
  • whoops, I was a bit quick. A fix for a similar issue with the smoothgl3 demo was applied. Could you see if that one was broken for you before r.1385 and now works for you with r.1385? Then a similar fix can help for the shapes demo.

    Thanks!

     
  • Trunk fixes the same error in smoothgl3 but shapes error remains. The prototype adds a "const ourGLchar **string" type. Why are you redefining the prototype for PFNGLSHADERSOURCEPROC in shapes.c? All those prototypes are defined in glext.h. Use them; if Windows doesn't have them, include glext.h in your sources.

     
  • I think this should all be fixed now (a while ago already), thanks for the report, comments, and help!

     
    • status: open --> closed-fixed