On Fri, 21 Aug 2015 23:24:54 -0700
Bryce Harrington <br...@osg.samsung.com> wrote:

> On Fri, Aug 21, 2015 at 11:06:02PM -0500, Derek Foreman wrote:
> > commit 89c49b3060a115e846ba1e7fbef94d14894244f2 changed the way we name
> > outputs, but it also added the new output names VIRTUAL and DSI.
> > 
> > These aren't available until libdrm 2.4.59

Hi,

sorry I forgot to check when those appeared.

> Ubuntu 14.04 appears to have 2.4.60.  Presumably other distros of that
> vintige will have similar.  So this looks like a reasonable dependency
> bump to me.
>  
> > Signed-off-by: Derek Foreman <der...@osg.samsung.com>
> 
> Reviewed-by: Bryce Harrington <br...@osg.samsung.com>

Alright. It would be very easy to just #ifdef these in case we wanted
to keep the libdrm requirement the same. I pushed your patch, and we
can fix it the other way if someone complains. (It's a 3 year bump, so
I'm not surprised if I need to fix it the other way.)
   decc965..faee330  master -> master


Thanks,
pq

> > ---
> >  configure.ac | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/configure.ac b/configure.ac
> > index 425f071..d70777d 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -171,7 +171,7 @@ AC_ARG_ENABLE(drm-compositor, [  
> > --enable-drm-compositor],,
> >  AM_CONDITIONAL(ENABLE_DRM_COMPOSITOR, test x$enable_drm_compositor = xyes)
> >  if test x$enable_drm_compositor = xyes; then
> >    AC_DEFINE([BUILD_DRM_COMPOSITOR], [1], [Build the DRM compositor])
> > -  PKG_CHECK_MODULES(DRM_COMPOSITOR, [libudev >= 136 libdrm >= 2.4.30 gbm 
> > mtdev >= 1.1.0])
> > +  PKG_CHECK_MODULES(DRM_COMPOSITOR, [libudev >= 136 libdrm >= 2.4.59 gbm 
> > mtdev >= 1.1.0])
> >    PKG_CHECK_MODULES(DRM_COMPOSITOR_GBM, [gbm >= 10.2],
> >                 [AC_DEFINE([HAVE_GBM_FD_IMPORT], 1, [gbm supports dmabuf 
> > import])],
> >                 [AC_MSG_WARN([gbm does not support dmabuf import, will omit 
> > that capability])])
> > -- 
> > 2.4.6

Attachment: pgpVOf7jhC6E_.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to