Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-21 Thread Ross Burton
On Tuesday, 20 November 2012 at 21:48, Brian Paul wrote: Personally, I trust Kilgard's GLUT more freeglut. Some versions of freeglut (hopefully fixed by now but I don't know) spend an inordinate amount of time in a InitSpaceBall() function, with a significant FPS hit. Another problem I've

Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-21 Thread Matt Turner
On Tue, Nov 20, 2012 at 1:52 PM, Ross Burton ross.bur...@intel.com wrote: On Tuesday, 20 November 2012 at 21:48, Brian Paul wrote: Personally, I trust Kilgard's GLUT more freeglut. Some versions of freeglut (hopefully fixed by now but I don't know) spend an inordinate amount of time in a

Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-21 Thread Burton, Ross
On 21 November 2012 08:27, Matt Turner matts...@gmail.com wrote: To be clear, Mesa's GLU was a thing, but it is not this thing: http://cgit.freedesktop.org/mesa/glu That's SGI's GLU. I hope you're not using Mesa's GLU. Ah, thanks. Yes, I'm using that repository. Ross

Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-20 Thread Burton, Ross
On 14 November 2012 15:38, Dan Nicholson dbn.li...@gmail.com wrote: This looks pretty good except that I think you need to temporarily add GLUT_CFLAGS to CFLAGS and GLUT_LIBS to LIBS so that the user specified prefix works for AC_CHECK*. On the other hand, at least mesa glut has a pkg-config

Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-20 Thread Kenneth Graunke
On 11/20/2012 08:57 AM, Burton, Ross wrote: On 14 November 2012 15:38, Dan Nicholson dbn.li...@gmail.com wrote: This looks pretty good except that I think you need to temporarily add GLUT_CFLAGS to CFLAGS and GLUT_LIBS to LIBS so that the user specified prefix works for AC_CHECK*. On the other

Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-20 Thread Brian Paul
On Tue, Nov 20, 2012 at 3:34 PM, Kenneth Graunke kenn...@whitecape.org wrote: On 11/20/2012 08:57 AM, Burton, Ross wrote: On 14 November 2012 15:38, Dan Nicholson dbn.li...@gmail.com wrote: This looks pretty good except that I think you need to temporarily add GLUT_CFLAGS to CFLAGS and

Re: [Mesa-dev] [PATCH] build: fix --without-glut

2012-11-14 Thread Dan Nicholson
On Nov 13, 2012 11:09 PM, Ross Burton ross.bur...@intel.com wrote: The argument --without-glut is transformed to --with-glut=no, which the logic wasn't handling at all so --without-glut didn't work. Rewrite the logic to handle the case where the value passed to --with-glut is no.

[Mesa-dev] [PATCH] build: fix --without-glut

2012-11-13 Thread Ross Burton
The argument --without-glut is transformed to --with-glut=no, which the logic wasn't handling at all so --without-glut didn't work. Rewrite the logic to handle the case where the value passed to --with-glut is no. Signed-off-by: Ross Burton ross.bur...@intel.com --- configure.ac | 26