Issue 48 in vrjuggler: Build broken on master since the GL3 merge

[vrju...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 48 by ryan.pav...@gmail.com: Build broken on master since the GL3
merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

This is my automated build - Ubuntu 11.04 x64.

modules/vrjuggler/vrj/Draw/OpenGL/DrawAxesFunctorsCore.cpp:41:23: fatal
error: GL3/gl3.h: No such file or directory
compilation terminated.

[vrju...@googlecode.com]

Comment #1 on issue 48 by mc...@iastate.edu: Build broken on master since

the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

Is GL3 installed on your system? It should be installed with your graphics
driver. If not then we will need to refactor somethings.

[vrju...@googlecode.com]

Comment #2 on issue 48 by ryan.pav...@gmail.com: Build broken on master

since the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

I don't see the include file - That machine has 270.41.06-0ubuntu1.2 of the
NVIDIA driver installed on it from the standard ubuntu package.

The hardware doesn't really do GL 3, IIRC, though that shouldn't keep it
from compiling (lspci calls it a nVidia Corporation NV45GL [Quadro FX
3400/4400] )

[vrju...@googlecode.com]

Comment #3 on issue 48 by doug.mcc...@gmail.com: Build broken on master

since the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

Well that is no good. I will dig into things and see what we need to do on
linux to make this build correctly.

[vrju...@googlecode.com]

Comment #4 on issue 48 by ryan.pav...@gmail.com: Build broken on master

since the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

http://www.opengl.org/registry/ suggests GL3/gl3.h was renamed to
GL/glcorearb.h with the release of OpenGL 4.3. I don't appear to have that
header installed either.

[vrju...@googlecode.com]

Comment #5 on issue 48 by doug.mcc...@gmail.com: Build broken on master

since the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

If you want to get your build going right away you can download the header:

http://www.opengl.org/registry/api/gl3.h

and place it in a GL3 directory on the system. I am still working on a
permanent solution.

[Todd Furlong]

GL3/gl3.h is supposed to get copied to the VR Juggler include directory as part of the build process, in which case it would be found on any system.  I think that's the part that is missing here.  However, as Doug points out the new header is glcorearb.h.  We'll need to come up with a long-term solution to this.

[Doug McCorkle]

On Tue, Jan 8, 2013 at 4:09 PM, Todd Furlong <furl...@gmail.com> wrote:
> GL3/gl3.h is supposed to get copied to the VR Juggler include directory as
> part of the build process, in which case it would be found on any system. I
> think that's the part that is missing here. However, as Doug points out the
> new header is glcorearb.h. We'll need to come up with a long-term solution
> to this.
>
>

During the merge of the gl3 branch the install of the header and
include directive was removed from the build on linux per a discussion
on the list.

https://code.google.com/p/vrjuggler/source/detail?r=445527a530085f39a2da355c10451238ad523bde

I thought (from my testing) that it was included with the graphics
drivers like gl.h. I believe I was wrong about that or must have
installed it myself on linux.

[Todd Furlong]

It's definitely not included everywhere.  I must have missed or misunderstood the discussion on its removal.  It doesn't need to be a part of the headers distributed in a VR Juggler install, but it does need to be found during the build.

In developing this code, we made a choice to avoid using GLEW (http://glew.sourceforge.net/), but maybe it is appropriate to do so and stop doing extension loading in VR Juggler.  Thoughts?

-Todd

[Ryan Pavlik]

I'm typically against re-inventing the wheel, and so would favor "outsourcing" to GLEW. There seems to be some criticism of it online, though I'm unsure of how much of it is still relevant to recent versions.  I do know I've seen a binary for it in a number of Windows apps.  Perhaps some useful info might be here: http://www.opengl.org/wiki/OpenGL_Loading_Library

[Doug McCorkle]

On Tue, Jan 8, 2013 at 4:21 PM, Todd Furlong <furl...@gmail.com> wrote:
> It's definitely not included everywhere. I must have missed or
> misunderstood the discussion on its removal. It doesn't need to be a part
> of the headers distributed in a VR Juggler install, but it does need to be
> found during the build.
>
> In developing this code, we made a choice to avoid using GLEW
> (http://glew.sourceforge.net/), but maybe it is appropriate to do so and
> stop doing extension loading in VR Juggler. Thoughts?
>

With a recent project we ran into a number of problems with glew. I
would prefer to stick with what we have and resolve how to get the new
ogl core profile header.

Doug

[vrju...@googlecode.com]

Comment #6 on issue 48 by mc...@iastate.edu: Build broken on master since

the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

Give this patch a try. I think this should resolve things.

Attachments:
0001-Changed-how-OpenGL-3-4-headers-are-found.-Currently.patch 445 KB

[vrju...@googlecode.com]

Comment #7 on issue 48 by mc...@iastate.edu: Build broken on master since

the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

Here is an additional patch addressing comments from Todd.

Attachments:
0001-Changed-how-OpenGL-3-4-headers-are-found.-Currently.patch 446 KB

[vrju...@googlecode.com]

Comment #8 on issue 48 by mc...@iastate.edu: Build broken on master since

the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

Patrick this can be closed.

[vrju...@googlecode.com]

Updates:
Status: Fixed
Owner: doug.mcc...@gmail.com
Labels: OpSys-Linux

Comment #9 on issue 48 by patrick....@gmail.com: Build broken on master

since the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

(No comment was entered for this change.)

[vrju...@googlecode.com]

Comment #10 on issue 48 by ryan.pav...@gmail.com: Build broken on master

since the GL3 merge
http://code.google.com/p/vrjuggler/issues/detail?id=48

I can confirm that this fixed the issue - when the commit hit my CI server
picked it up and the build succeeded.