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 Paul: 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 u

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 Paul: 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 u

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 Paul wrote: On 02/26/2013 10:09 AM, Jordan Justen wrote: On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciat

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 wrote: > On 02/26/2013 10:09 AM, Jordan Justen wrote: >> >> On Sat, Feb 23, 2013 at 7: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 bu

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 Justen On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul wrote: This series removes the dependencies on t

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 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. I'm getting a build error: G

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 wrote: > On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul wrote: > > This series re

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

2013-02-26 Thread Andreas Boll
2013/2/26 Brian Paul : > 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 FEATURE_{GL,ES1,ES2} > >

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 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 updating main/git_s

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 maili

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

2013-02-25 Thread Eric Anholt
Ian Romanick writes: > On 02/25/2013 05:17 PM, Brian Paul wrote: >> On 02/25/2013 11:10 AM, Jordan Justen wrote: >>> Reviewed-by: Jordan Justen >>> >>> On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul wrote: This series removes the dependencies on the mfeatures.h file and the file itse

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 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 Justen On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul wrote: >>

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 Justen On Sat, Feb 23, 2013 at 7:29 AM, Brian Paul wrote: This series removes the dependencies on the mfeatures.h file and the file itself. I

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 Justen On Sat, Feb 23, 2013 at 7: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

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 Justen On Sat, Feb 23, 2013 at 7: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. 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 and

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

2013-02-25 Thread Jordan Justen
Reviewed-by: Jordan Justen On Sat, Feb 23, 2013 at 7: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. Do you have a public repo where you push c

[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