[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