<div dir="ltr">Hi Bryce,<div><br><div>basically it's not my project :-), i'm working for a big german tier one and we are using weston along with ivi-shell,</div><div>where ivi environment controls the display in first place, not the user.</div></div><div>Since we are also dependent on other suppliers we cannot update everything in our linux distribution as far as we would like, but this current thread shows us that we need to keep an eye also on libdrm version and force to move our suppliers also in right direction :-)</div><div><br></div><div>Actually in our current development we are facing some missing functionality to fulfill all of our use-case but I will not start a discussion here I will create a separate thread to describe those and collect some ideas.</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-25 1:15 GMT+02:00 Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Aug 24, 2015 at 01:06:45PM +0200, Eugen Friedrich wrote:<br>
> Hello dear community,<br>
><br>
> in our current project we are trying to follow the weston master branch as<br>
> much as possible,<br>
<br>
</span>Hi Eugen, can you tell us more about your project?<br>
<span class="HOEnZb"><font color="#888888"><br>
Bryce<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> recently introduced dependency to libdrm 2.4.59 will prevent us to use<br>
> weston 1.9,<br>
> The patches for naming correction are good and important.<br>
> so we would love to see a missing defines in the corresponding #ifdef<br>
> statement.<br>
><br>
> Are display name defines the only reason to add a dependency to 2.4.59?<br>
><br>
> 2015-08-24 10:52 GMT+02:00 Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>>:<br>
><br>
> > On Fri, 21 Aug 2015 23:24:54 -0700<br>
> > Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>> wrote:<br>
> ><br>
> > > On Fri, Aug 21, 2015 at 11:06:02PM -0500, Derek Foreman wrote:<br>
> > > > commit 89c49b3060a115e846ba1e7fbef94d14894244f2 changed the way we name<br>
> > > > outputs, but it also added the new output names VIRTUAL and DSI.<br>
> > > ><br>
> > > > These aren't available until libdrm 2.4.59<br>
> ><br>
> > Hi,<br>
> ><br>
> > sorry I forgot to check when those appeared.<br>
> ><br>
> > > Ubuntu 14.04 appears to have 2.4.60.  Presumably other distros of that<br>
> > > vintige will have similar.  So this looks like a reasonable dependency<br>
> > > bump to me.<br>
> > ><br>
> > > > Signed-off-by: Derek Foreman <<a href="mailto:derekf@osg.samsung.com">derekf@osg.samsung.com</a>><br>
> > ><br>
> > > Reviewed-by: Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>><br>
> ><br>
> > Alright. It would be very easy to just #ifdef these in case we wanted<br>
> > to keep the libdrm requirement the same. I pushed your patch, and we<br>
> > can fix it the other way if someone complains. (It's a 3 year bump, so<br>
> > I'm not surprised if I need to fix it the other way.)<br>
> >    decc965..faee330  master -> master<br>
> ><br>
> ><br>
> > Thanks,<br>
> > pq<br>
> ><br>
> > > > ---<br>
> > > >  <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 2 +-<br>
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> > > ><br>
> > > > diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> > > > index 425f071..d70777d 100644<br>
> > > > --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> > > > +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> > > > @@ -171,7 +171,7 @@ AC_ARG_ENABLE(drm-compositor, [<br>
> > --enable-drm-compositor],,<br>
> > > >  AM_CONDITIONAL(ENABLE_DRM_COMPOSITOR, test x$enable_drm_compositor =<br>
> > xyes)<br>
> > > >  if test x$enable_drm_compositor = xyes; then<br>
> > > >    AC_DEFINE([BUILD_DRM_COMPOSITOR], [1], [Build the DRM compositor])<br>
> > > > -  PKG_CHECK_MODULES(DRM_COMPOSITOR, [libudev >= 136 libdrm >= 2.4.30<br>
> > gbm mtdev >= 1.1.0])<br>
> > > > +  PKG_CHECK_MODULES(DRM_COMPOSITOR, [libudev >= 136 libdrm >= 2.4.59<br>
> > gbm mtdev >= 1.1.0])<br>
> > > >    PKG_CHECK_MODULES(DRM_COMPOSITOR_GBM, [gbm >= 10.2],<br>
> > > >                 [AC_DEFINE([HAVE_GBM_FD_IMPORT], 1, [gbm supports<br>
> > dmabuf import])],<br>
> > > >                 [AC_MSG_WARN([gbm does not support dmabuf import, will<br>
> > omit that capability])])<br>
> > > > --<br>
> > > > 2.4.6<br>
> ><br>
> > _______________________________________________<br>
> > wayland-devel mailing list<br>
> > <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> > <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
> ><br>
> ><br>
</div></div></blockquote></div><br></div>