[Mesa-dev] [PATCH] configure.ac: don't require DRM and GBM for surfaceless egl platform

[Marek Olšák]

On Mon, Oct 17, 2016 at 7:10 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 17 October 2016 at 17:47, Marek Olšák <maraeo at gmail.com> wrote:
>> From: Deepak Sharma <Deepak.Sharma at amd.com>
>>
>> This allows building radeonsi driver for surfaceless platform.
>>
>> Signed-off-by: Deepak Sharma <Deepak.Sharma at amd.com>
>> Signed-off-by: Marek Olšák <marek.olsak at amd.com>
> As I "predicted" from the beginning adding require_egl_drm() is a hack
> which will inspire even more hacks.
>
> For example 98cebc913cd7ad07ce9699b5f63d72fe3969c851 and now this one.
> Please _drop_ these all together.
>
> I realise what you're trying to be extra careful on behalf of the
> user, but this is not the way to do it. Let's drop the "interesting"
> enable-gbm auto-detection enabling it by default or toggling it on
> supported platforms - !cygwin/windows and add the drm egl-platform if
> gbm is enabled. This way your worries will be resolved, plus things
> will be shorter and easier to untangle :-)

While I understand where your criticism is coming from, this is not a
hack to me. It's really just an improvement to already-good code. ;)

Marek