Re: [PATCH weston] build: Require newer version of libdrm
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 br...@osg.samsung.com: 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 ppaala...@gmail.com: 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 ___ 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
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 br...@osg.samsung.com: 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 ppaala...@gmail.com: 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 ___ 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
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 ppaala...@gmail.com: 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 ___ 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
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 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
On Mon, 24 Aug 2015 13:06:45 +0200 Eugen Friedrich fried...@gmail.com 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 ppaala...@gmail.com: 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 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
On Mon, 24 Aug 2015 14:16:17 +0300 Pekka Paalanen ppaala...@gmail.com wrote: On Mon, 24 Aug 2015 13:06:45 +0200 Eugen Friedrich fried...@gmail.com 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
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 ppaala...@gmail.com: 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 ___ 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
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 der...@osg.samsung.com Reviewed-by: Bryce Harrington br...@osg.samsung.com --- 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