Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-27 Thread Brian Paul
On 02/26/2013 10:16 AM, Andreas Boll wrote: 2013/2/26 Brian Paulbri...@vmware.com: OK, I have a git tree on fd.o with this patch series (plus updates to intel_screen.c and configure.ac). With your configure.ac patch API_DEFINES seems now unused. But from what I can see

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Brian Paul
OK, I have a git tree on fd.o with this patch series (plus updates to intel_screen.c and configure.ac). git clone git://people.freedesktop.org/~brianp/mesa.git git check-out -b remove-mfeatures remotes/origin/remove-mfeatures -Brian ___ mesa-dev

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Jordan Justen
On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul bri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of this series to double-check my work. I'm getting a build error: GENmain/get_hash.h

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Andreas Boll
2013/2/26 Brian Paul bri...@vmware.com: OK, I have a git tree on fd.o with this patch series (plus updates to intel_screen.c and configure.ac). With your configure.ac patch API_DEFINES seems now unused. But from what I can see src/gallium/targets/egl-static/egl_st.c used it to define

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Aaron Watry
Same error here. Configuration: ./autogen.sh --enable-texture-float --enable-opencl --with-gallium-drivers=r600 --with-dri-drivers=radeon --prefix=/usr/local --Aaron On Tue, Feb 26, 2013 at 11:09 AM, Jordan Justen jljus...@gmail.com wrote: On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Brian Paul
On 02/26/2013 10:09 AM, Jordan Justen wrote: On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of this series to double-check my work. I'm getting a

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Kenneth Graunke
On 02/25/2013 06:48 PM, Brian Paul wrote: On 02/25/2013 06:57 PM, Ian Romanick wrote: On 02/25/2013 05:17 PM, Brian Paul wrote: On 02/25/2013 11:10 AM, Jordan Justen wrote: Reviewed-by: Jordan Justenjordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote:

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Jordan Justen
On Tue, Feb 26, 2013 at 10:16 AM, Brian Paul bri...@vmware.com wrote: On 02/26/2013 10:09 AM, Jordan Justen wrote: On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Brian Paul
On 02/26/2013 11:58 AM, Jordan Justen wrote: On Tue, Feb 26, 2013 at 10:16 AM, Brian Paulbri...@vmware.com wrote: On 02/26/2013 10:09 AM, Jordan Justen wrote: On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-26 Thread Brian Paul
On 02/26/2013 10:16 AM, Andreas Boll wrote: 2013/2/26 Brian Paulbri...@vmware.com: OK, I have a git tree on fd.o with this patch series (plus updates to intel_screen.c and configure.ac). With your configure.ac patch API_DEFINES seems now unused. But from what I can see

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Jordan Justen
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul bri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of this series to double-check my work. Do

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Ian Romanick
On 02/23/2013 07:29 AM, Brian Paul wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of this series to double-check my work. The last thing left to do is remove the -DFEATURE_GL/ES1/ES2 stuff from the autoconf

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Brian Paul
On 02/25/2013 11:10 AM, Jordan Justen wrote: Reviewed-by: Jordan Justenjordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Ian Romanick
On 02/25/2013 05:17 PM, Brian Paul wrote: On 02/25/2013 11:10 AM, Jordan Justen wrote: Reviewed-by: Jordan Justenjordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on the mfeatures.h file and the file itself.

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Brian Paul
On 02/25/2013 06:57 PM, Ian Romanick wrote: On 02/25/2013 05:17 PM, Brian Paul wrote: On 02/25/2013 11:10 AM, Jordan Justen wrote: Reviewed-by: Jordan Justenjordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Matt Turner
On Mon, Feb 25, 2013 at 6:48 PM, Brian Paul bri...@vmware.com wrote: On 02/25/2013 06:57 PM, Ian Romanick wrote: On 02/25/2013 05:17 PM, Brian Paul wrote: On 02/25/2013 11:10 AM, Jordan Justen wrote: Reviewed-by: Jordan Justenjordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM,

Re: [Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-25 Thread Eric Anholt
Ian Romanick i...@freedesktop.org writes: On 02/25/2013 05:17 PM, Brian Paul wrote: On 02/25/2013 11:10 AM, Jordan Justen wrote: Reviewed-by: Jordan Justenjordan.l.jus...@intel.com On Sat, Feb 23, 2013 at 7:29 AM, Brian Paulbri...@vmware.com wrote: This series removes the dependencies on

[Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-23 Thread Brian Paul
This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of this series to double-check my work. The last thing left to do is remove the -DFEATURE_GL/ES1/ES2 stuff from the autoconf and scons files. -Brian