[Mesa-dev] [PATCH 1/6] configure.ac: remove redundant option --enable-gallium-egl

[Chia-I Wu]

On Thu, Jun 16, 2011 at 10:20 AM, Marek Olšák <maraeo at gmail.com> wrote:
> On Thu, Jun 16, 2011 at 1:21 AM, Chia-I Wu <olvaffe at gmail.com> wrote:
>> On Thu, Jun 16, 2011 at 7:04 AM, Marek Olšák <maraeo at gmail.com> wrote:
>>> On Tue, Jun 14, 2011 at 10:35 PM, Benjamin Franzke
>>> <benjaminfranzke at googlemail.com> wrote:
>>>> Removing this flag seems right to me, but always building the egl
>>>> state tracker when gallium and egl is enabled not.
>>>> So when --with-state-trackers is also removed, we'd need a new
>>>> --with-egl-drivers=auto|gallium,dri2,glx or so.
>>>
>>> Isn't --with-egl-platforms good enough to select among fbdev, wayland,
>>> one-day-maybe dri, ...? I don't see a reason for another option. You
>>> can choose between dri and xlib (st/glx) already using
>>> --with-driver=dri|xlib.
>> OpenGL can be supported via both egl_dri2 and egl_gallium EGL drivers.
>>  The former is generally preferred for various reasons.  However, when
>> both drivers exist, there is no automatic way to make EGL chooses the
>> former.  Thus it is desirable to be able to disable egl_gallium in the
>> first place.
>
> Why not to use egl_dri2 for all drivers, the classic and gallium ones?
>
> Why not to build a single fully-replaceable binary for a driver, like
> classic swrast_dri.so can be replaced by swrastg_dri.so from Gallium?
> Like radeon_drv.so from xf86-video-ati can be replaced by r300_drv.so
> from Gallium?
>
> I thought it had been agreed long ago that Gallium is a private
> unstable interface inside drivers and shouldn't be exposed publicly in
> any way. With that, I see no reason for egl_gallium.so to exist.
egl_gallium supports OpenVG and some EGL extensions not supported by
egl_dri2.  SCons's build of egl_gallium does not expose any internal
interface.  I can(/should have) adjust autoconf-based build not to
too.

Maybe we can have egl_dri2 enabled and egl_gallium disabled when
--enable-egl, and has the option to enable egl_gallium?  Then maybe
have --enable-openvg depend on egl_gallium?

> Marek
>

-- 
olv at LunarG.com