[PATCH] configure.ac: fix broken compilation when configure with --disable-egl option

Kristian Høgsberg hoegsberg at gmail.com
Wed Aug 28 22:30:41 PDT 2013


On Wed, Aug 28, 2013 at 11:57:57AM +0200, Armin K. wrote:
> On 08/28/2013 11:22 AM, Samuel Iglesias Gonsalvez wrote:
> > Fix bug 67561: "configure with --disable-egl option breaks fbdev backend
> > compilation"
> > 
> > https://bugs.freedesktop.org/show_bug.cgi?id=67561
> > 
> > Signed-off-by: Samuel Iglesias Gonsalvez <siglesias at igalia.com>
> > ---
> >  configure.ac | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/configure.ac b/configure.ac
> > index 156237f..0470768 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -190,7 +190,7 @@ AM_CONDITIONAL([ENABLE_FBDEV_COMPOSITOR],
> >                 [test x$enable_fbdev_compositor = xyes])
> >  AS_IF([test x$enable_fbdev_compositor = xyes], [
> >    AC_DEFINE([BUILD_FBDEV_COMPOSITOR], [1], [Build the fbdev compositor])
> > -  PKG_CHECK_MODULES([FBDEV_COMPOSITOR], [libudev >= 136 mtdev >= 1.1.0])
> > +  PKG_CHECK_MODULES([FBDEV_COMPOSITOR], [libudev >= 136 mtdev >= 1.1.0 libdrm >= 2.4.30])
> >  ])
> >  
> >  AC_ARG_ENABLE([rdp-compositor], [  --enable-rdp-compositor],,
> > 
> 
> I've sent a bit cleaner fix back in July. You don't actualy need libdrm
> for fbdev-backend
> 
> http://lists.freedesktop.org/archives/wayland-devel/2013-July/010435.html

Sorry, I missed that.  I think Samuels fix is better since it's really
just going back to how it (somewhat accidentally) worked before.  The
real fix will be to move set/drop drm master responsibility into
weston-launch, and in the meantime I prefer a smaller change.

Kristian


More information about the wayland-devel mailing list