[Mesa-dev] [PATCH 2/2] configure.ac: enable building the EGL DRM platform by default
emil.l.velikov at gmail.com
Tue May 26 18:21:49 PDT 2015
./autogen.sh --with-gallium-drivers=radeonsi --enable-egl --enable-gbm
seems to work fine here. Although... if I drop the --enable-gbm it
complains "EGL platform drm needs gbm", which is an entirely different
kettle of fish.
Fwiw I would opt to just enable gbm on supported platforms (everything
but Win/Mac) and then add drm to the list, based on --enable-gbm. If you
don't comfortable with it, I'll send patches as with follow up commit(s).
On 26/05/15 21:46, Marek Olšák wrote:
> No, it doesn't work. If I just add "drm" to the defaults and not use
> --with-egl-platforms in the terminal, it fails with the message that
> radeonsi requires drm. :)
> On Tue, May 26, 2015 at 10:30 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> On 26/05/15 10:57, Marek Olšák wrote:
>>> From: Marek Olšák <marek.olsak at amd.com>
>>> This is required by radeonsi.
>>> The code was moved because with_egl_platforms was used before
>>> the default was set.
>>> configure.ac | 21 ++++++++++-----------
>>> 1 file changed, 10 insertions(+), 11 deletions(-)
>>> diff --git a/configure.ac b/configure.ac
>>> index 84b404e..dbe590f 100644
>>> --- a/configure.ac
>>> +++ b/configure.ac
>>> @@ -763,6 +763,16 @@ AC_ARG_ENABLE([egl],
>>> [disable EGL library @<:@default=enabled@:>@])],
>>> + [AS_HELP_STRING([--with-egl-platforms@<:@=DIRS...@:>@],
>>> + [comma delimited native platforms libEGL supports, e.g.
>>> + "x11,drm" @<:@default=auto@:>@])],
>>> + [with_egl_platforms="$withval"],
>>> + [if test "x$enable_egl" = xyes; then
>>> + with_egl_platforms="x11,drm"
>>> + else
>>> + with_egl_platforms=""
>>> + fi])
>> Can we just add drm without moving the hole hunk ? From a vague look it
>> should just work.
More information about the mesa-dev