Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-30 Thread Emil Velikov
On 28 May 2015 at 21:48, Matt Turner wrote: > On Thu, May 28, 2015 at 5:07 AM, Emil Velikov > wrote: >> On 27/05/15 16:59, Matt Turner wrote: >>> On Wed, May 27, 2015 at 4:53 AM, Emil Velikov >>> wrote: On 27 May 2015 at 11:23, Dave Airlie wrote: >> Wow, I hadn't expected such a hate

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-28 Thread Matt Turner
On Thu, May 28, 2015 at 5:07 AM, Emil Velikov wrote: > On 27/05/15 16:59, Matt Turner wrote: >> On Wed, May 27, 2015 at 4:53 AM, Emil Velikov >> wrote: >>> On 27 May 2015 at 11:23, Dave Airlie wrote: > Wow, I hadn't expected such a hateful comment on GLES1. > > Does anyone else want

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-28 Thread Emil Velikov
On 27/05/15 16:59, Matt Turner wrote: > On Wed, May 27, 2015 at 4:53 AM, Emil Velikov > wrote: >> On 27 May 2015 at 11:23, Dave Airlie wrote: Wow, I hadn't expected such a hateful comment on GLES1. Does anyone else want to convince me that GLES1 should burn in hell? >>> >>> So I d

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-27 Thread Matt Turner
On Wed, May 27, 2015 at 4:53 AM, Emil Velikov wrote: > On 27 May 2015 at 11:23, Dave Airlie wrote: >>> Wow, I hadn't expected such a hateful comment on GLES1. >>> >>> Does anyone else want to convince me that GLES1 should burn in hell? >> >> So I dug around, >> >> commit 4c06853833996d990eb76b195

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-27 Thread Marek Olšák
The cost of shipping GLES1 is 27KB for libGLESv1_CM.so.1.1.0, so I honestly think there shouldn't even be an option to disable it. It's not something I particularly care about though. Marek On Wed, May 27, 2015 at 1:53 PM, Emil Velikov wrote: > On 27 May 2015 at 11:23, Dave Airlie wrote: >>> Wo

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-27 Thread Emil Velikov
On 27 May 2015 at 11:23, Dave Airlie wrote: >> Wow, I hadn't expected such a hateful comment on GLES1. >> >> Does anyone else want to convince me that GLES1 should burn in hell? > > So I dug around, > > commit 4c06853833996d990eb76b195ca5d6838c6f3d6b > Author: Adam Jackson > Date: Wed May 8 18:

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-27 Thread Dave Airlie
> Wow, I hadn't expected such a hateful comment on GLES1. > > Does anyone else want to convince me that GLES1 should burn in hell? So I dug around, commit 4c06853833996d990eb76b195ca5d6838c6f3d6b Author: Adam Jackson Date: Wed May 8 18:03:21 2013 -0400 Switch to Mesa master (pre 9.2)

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-27 Thread Marek Olšák
Wow, I hadn't expected such a hateful comment on GLES1. Does anyone else want to convince me that GLES1 should burn in hell? Marek On Wed, May 27, 2015 at 7:01 AM, Dave Airlie wrote: > On 26 May 2015 at 22:08, Marek Olšák wrote: >> It's mainly for distributions. You can turn it off, but it won

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Dave Airlie
On 26 May 2015 at 22:08, Marek Olšák wrote: > It's mainly for distributions. You can turn it off, but it won't make > the build faster, because all code is shared. we (fedora/rhel) don't enable GLES1, and hope nobody else wants to either. gles2 is fine, leave gles1 die. Dave. ___

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Tapani
On 05/26/2015 03:08 PM, Marek Olšák wrote: It's mainly for distributions. You can turn it off, but it won't make the build faster, because all code is shared. Right, then seems good move to me. Marek On Tue, May 26, 2015 at 1:26 PM, Tapani Pälli wrote: On 05/26/2015 01:57 PM, Marek Olšák w

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Emil Velikov
On 26/05/15 10:57, Marek Olšák wrote: > From: Marek Olšák > > No particular reason other than most people should always build this. Yes please. Personally I've not seen any noticable increase in the build time, plus anyone is welcome to disable it as they please. If you need to go for the defaul

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Ian Romanick
On 05/26/2015 05:08 AM, Marek Olšák wrote: > It's mainly for distributions. You can turn it off, but it won't make > the build faster, because all code is shared. If this could be used as a step towards eventually eliminating some configure options altogether, I'm definitely in favor > Marek

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Matt Turner
On Tue, May 26, 2015 at 3:57 AM, Marek Olšák wrote: > From: Marek Olšák > > No particular reason other than most people should always build this. > --- No preference from me, but the patch does what it claims: Reviewed-by: Matt Turner ___ mesa-dev ma

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Marek Olšák
It's mainly for distributions. You can turn it off, but it won't make the build faster, because all code is shared. Marek On Tue, May 26, 2015 at 1:26 PM, Tapani Pälli wrote: > > On 05/26/2015 01:57 PM, Marek Olšák wrote: >> >> From: Marek Olšák >> >> No particular reason other than most people

Re: [Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Tapani Pälli
On 05/26/2015 01:57 PM, Marek Olšák wrote: From: Marek Olšák No particular reason other than most people should always build this. Is there really a reason for most people to build gles1 support? IIRC SurfaceFlinger uses it but haven't seen much other users. Maybe it should be enabled by d

[Mesa-dev] [PATCH 1/2] configure.ac: enable building GLES1 and GLES2 by default

2015-05-26 Thread Marek Olšák
From: Marek Olšák No particular reason other than most people should always build this. --- configure.ac | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/configure.ac b/configure.ac index 5594548..84b404e 100644 --- a/configure.ac +++ b/configure.ac @@ -714,15 +71