Re: [PATCH weston] build: Require newer version of libdrm

2015-08-25 Thread Bryce Harrington
On Tue, Aug 25, 2015 at 09:54:58PM +0200, Eugen Friedrich wrote:
> Hi Bryce,
> 
> basically it's not my project :-), i'm working for a big german tier one
> and we are using weston along with ivi-shell,
> where ivi environment controls the display in first place, not the user.

Gotcha, sounds intriguing.

> 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 :-)

Yep, that and invariably the kernel, and mesa or whatever you're using
for GL support.
 
> 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.

Thanks, that would be useful information.  Also, if you guys have
patches you're carrying locally or plan to develop, it'd be great to see
them.
 
Bryce

> 2015-08-25 1:15 GMT+02:00 Bryce Harrington :
> 
> > On Mon, Aug 24, 2015 at 01:06:45PM +0200, Eugen Friedrich wrote:
> > > Hello dear community,
> > >
> > > in our current project we are trying to follow the weston master branch
> > as
> > > much as possible,
> >
> > Hi Eugen, can you tell us more about your project?
> >
> > Bryce
> >
> > > recently introduced dependency to libdrm 2.4.59 will prevent us to use
> > > weston 1.9,
> > > The patches for naming correction are good and important.
> > > so we would love to see a missing defines in the corresponding #ifdef
> > > statement.
> > >
> > > Are display name defines the only reason to add a dependency to 2.4.59?
> > >
> > > 2015-08-24 10:52 GMT+02:00 Pekka Paalanen :
> > >
> > > > On Fri, 21 Aug 2015 23:24:54 -0700
> > > > Bryce Harrington  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 
> > > > >
> > > > > Reviewed-by: Bryce Harrington 
> > > >
> > > > 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
> > > >
> > > > ___
> > > > wayland-devel mailing list
> > > > wayland-devel@lists.freedesktop.org
> > > > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> > > >
> > > >
> >
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-25 Thread Eugen Friedrich
Hi Bryce,

basically it's not my project :-), i'm working for a big german tier one
and we are using weston along with ivi-shell,
where ivi environment controls the display in first place, not the user.
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 :-)

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.

2015-08-25 1:15 GMT+02:00 Bryce Harrington :

> On Mon, Aug 24, 2015 at 01:06:45PM +0200, Eugen Friedrich wrote:
> > Hello dear community,
> >
> > in our current project we are trying to follow the weston master branch
> as
> > much as possible,
>
> Hi Eugen, can you tell us more about your project?
>
> Bryce
>
> > recently introduced dependency to libdrm 2.4.59 will prevent us to use
> > weston 1.9,
> > The patches for naming correction are good and important.
> > so we would love to see a missing defines in the corresponding #ifdef
> > statement.
> >
> > Are display name defines the only reason to add a dependency to 2.4.59?
> >
> > 2015-08-24 10:52 GMT+02:00 Pekka Paalanen :
> >
> > > On Fri, 21 Aug 2015 23:24:54 -0700
> > > Bryce Harrington  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 
> > > >
> > > > Reviewed-by: Bryce Harrington 
> > >
> > > 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
> > >
> > > ___
> > > wayland-devel mailing list
> > > wayland-devel@lists.freedesktop.org
> > > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> > >
> > >
>
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Bryce Harrington
On Mon, Aug 24, 2015 at 01:06:45PM +0200, Eugen Friedrich wrote:
> Hello dear community,
> 
> in our current project we are trying to follow the weston master branch as
> much as possible,

Hi Eugen, can you tell us more about your project?

Bryce

> recently introduced dependency to libdrm 2.4.59 will prevent us to use
> weston 1.9,
> The patches for naming correction are good and important.
> so we would love to see a missing defines in the corresponding #ifdef
> statement.
> 
> Are display name defines the only reason to add a dependency to 2.4.59?
> 
> 2015-08-24 10:52 GMT+02:00 Pekka Paalanen :
> 
> > On Fri, 21 Aug 2015 23:24:54 -0700
> > Bryce Harrington  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 
> > >
> > > Reviewed-by: Bryce Harrington 
> >
> > 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
> >
> > ___
> > wayland-devel mailing list
> > wayland-devel@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> >
> >
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Pekka Paalanen
On Mon, 24 Aug 2015 14:16:17 +0300
Pekka Paalanen  wrote:

> On Mon, 24 Aug 2015 13:06:45 +0200
> Eugen Friedrich  wrote:
> 
> > Hello dear community,
> > 
> > in our current project we are trying to follow the weston master branch as
> > much as possible,
> > recently introduced dependency to libdrm 2.4.59 will prevent us to use
> > weston 1.9,
> > The patches for naming correction are good and important.
> > so we would love to see a missing defines in the corresponding #ifdef
> > statement.
> 
> Heh, that came even faster than I expected. :-)
> 
> > Are display name defines the only reason to add a dependency to 2.4.59?
> 
> Yes, as far as I know.
> 

By the way, once Weston's atomic KMS feature lands in full, we will
probably bump the libdrm version requirement to at least 2.4.62. I
think this is quite likely to happen for Weston 1.10 (i.e. during
1.9.90).


Thanks,
pq


pgpbrckOsdGDr.pgp
Description: OpenPGP digital signature
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Pekka Paalanen
On Mon, 24 Aug 2015 13:06:45 +0200
Eugen Friedrich  wrote:

> Hello dear community,
> 
> in our current project we are trying to follow the weston master branch as
> much as possible,
> recently introduced dependency to libdrm 2.4.59 will prevent us to use
> weston 1.9,
> The patches for naming correction are good and important.
> so we would love to see a missing defines in the corresponding #ifdef
> statement.

Heh, that came even faster than I expected. :-)

> Are display name defines the only reason to add a dependency to 2.4.59?

Yes, as far as I know.


I'll whip up a patch.


Thanks,
pq

> 2015-08-24 10:52 GMT+02:00 Pekka Paalanen :
> 
> > On Fri, 21 Aug 2015 23:24:54 -0700
> > Bryce Harrington  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 
> > >
> > > Reviewed-by: Bryce Harrington 
> >
> > 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
> >


pgp5s5JFUirF_.pgp
Description: OpenPGP digital signature
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Eugen Friedrich
Hello dear community,

in our current project we are trying to follow the weston master branch as
much as possible,
recently introduced dependency to libdrm 2.4.59 will prevent us to use
weston 1.9,
The patches for naming correction are good and important.
so we would love to see a missing defines in the corresponding #ifdef
statement.

Are display name defines the only reason to add a dependency to 2.4.59?

2015-08-24 10:52 GMT+02:00 Pekka Paalanen :

> On Fri, 21 Aug 2015 23:24:54 -0700
> Bryce Harrington  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 
> >
> > Reviewed-by: Bryce Harrington 
>
> 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
>
> ___
> wayland-devel mailing list
> wayland-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
>
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-24 Thread Pekka Paalanen
On Fri, 21 Aug 2015 23:24:54 -0700
Bryce Harrington  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 
> 
> Reviewed-by: Bryce Harrington 

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


pgpVOf7jhC6E_.pgp
Description: OpenPGP digital signature
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH weston] build: Require newer version of libdrm

2015-08-21 Thread Bryce Harrington
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

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 

Reviewed-by: Bryce Harrington 
> ---
>  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
> 
> ___
> wayland-devel mailing list
> wayland-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


[PATCH weston] build: Require newer version of libdrm

2015-08-21 Thread Derek Foreman
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

Signed-off-by: Derek Foreman 
---
 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

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