From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from arroyo.ext.ti.com (arroyo.ext.ti.com [192.94.94.40]) by arago-project.org (Postfix) with ESMTPS id 2F3A452972 for ; Tue, 30 Sep 2014 11:22:08 +0000 (UTC) Received: from dbdlxv05.itg.ti.com ([172.24.171.60]) by arroyo.ext.ti.com (8.13.7/8.13.7) with ESMTP id s8UBM6HY029612 for ; Tue, 30 Sep 2014 06:22:07 -0500 Received: from DBDE72.ent.ti.com (dbdmailx.itg.ti.com [172.24.171.97]) by dbdlxv05.itg.ti.com (8.14.3/8.13.8) with ESMTP id s8UBM4s7012713 for ; Tue, 30 Sep 2014 16:52:04 +0530 Received: from [172.24.159.120] (172.24.159.120) by DBDE72.ent.ti.com (172.24.171.97) with Microsoft SMTP Server (TLS) id 14.3.174.1; Tue, 30 Sep 2014 16:52:03 +0530 Message-ID: <542A925B.5080402@ti.com> Date: Tue, 30 Sep 2014 16:52:03 +0530 From: Karthik Ramanan User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 MIME-Version: 1.0 To: Denys Dmytriyenko References: <1409928767-11517-1-git-send-email-a0393906@ti.com> <1409928767-11517-4-git-send-email-a0393906@ti.com> <20140926184907.GE6303@edge> In-Reply-To: <20140926184907.GE6303@edge> X-Originating-IP: [172.24.159.120] Cc: meta-arago@arago-project.org Subject: Re: [for master] [PATCH v2 3/4] wayland: add support for version 1.3.0 for drm backend X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 30 Sep 2014 11:22:08 -0000 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit On 27-Sep-14 12:19 AM, Denys Dmytriyenko wrote: > On Fri, Sep 05, 2014 at 08:22:46PM +0530, Karthik Ramanan wrote: >> For DRM based devices, it is desired to stick with version 1.3.0 >> due to some known issues and limitations with omapdrm and other >> dependent foundation software >> >> Signed-off-by: Karthik Ramanan >> --- >> .../recipes-graphics/wayland/wayland.inc | 5 ++++ >> .../recipes-graphics/wayland/wayland_1.3.0.bb | 32 ++++++++++------------ >> 2 files changed, 20 insertions(+), 17 deletions(-) >> create mode 100644 meta-arago-extras/recipes-graphics/wayland/wayland.inc >> >> diff --git a/meta-arago-extras/recipes-graphics/wayland/wayland.inc b/meta-arago-extras/recipes-graphics/wayland/wayland.inc >> new file mode 100644 >> index 0000000..8eb445d >> --- /dev/null >> +++ b/meta-arago-extras/recipes-graphics/wayland/wayland.inc >> @@ -0,0 +1,5 @@ >> +HOMEPAGE = "http://wayland.freedesktop.org/" >> +LICENSE = "MIT" >> +LIC_FILES_CHKSUM = "file://COPYING;md5=1d4476a7d98dd5691c53d4d43a510c72" >> + >> +inherit autotools pkgconfig >> diff --git a/meta-arago-extras/recipes-graphics/wayland/wayland_1.3.0.bb b/meta-arago-extras/recipes-graphics/wayland/wayland_1.3.0.bb >> index 212e08a..f40a98b 100644 >> --- a/meta-arago-extras/recipes-graphics/wayland/wayland_1.3.0.bb >> +++ b/meta-arago-extras/recipes-graphics/wayland/wayland_1.3.0.bb >> @@ -1,20 +1,4 @@ >> -SUMMARY = "Wayland, a protocol between a compositor and clients" >> -DESCRIPTION = "Wayland is a protocol for a compositor to talk to its clients \ >> -as well as a C library implementation of that protocol. The compositor can be \ >> -a standalone display server running on Linux kernel modesetting and evdev \ >> -input devices, an X application, or a wayland client itself. The clients can \ >> -be traditional applications, X servers (rootless or fullscreen) or other \ >> -display servers." >> -HOMEPAGE = "http://wayland.freedesktop.org" >> -LICENSE = "MIT" >> -LIC_FILES_CHKSUM = "file://COPYING;md5=1d4476a7d98dd5691c53d4d43a510c72 \ >> - file://src/wayland-server.c;endline=21;md5=079ae21dbf98ada52ec23744851b0a5c" >> - >> -SRC_URI = "http://wayland.freedesktop.org/releases/${BPN}-${PV}.tar.xz" >> -SRC_URI[md5sum] = "d16d27081e0871de82d08840c2f133fc" >> -SRC_URI[sha256sum] = "2e817685f68a26acd19964d69ddbc4549ba5412114ad95e1a9f5934cce470d6e" >> - >> -SRC_URI_append_class-native = " file://just-scanner.patch" >> +require wayland.inc >> >> inherit autotools pkgconfig >> >> @@ -27,6 +11,19 @@ DEPENDS = "expat libffi wayland-native" >> EXTRA_OECONF_virtclass-native = "--disable-documentation" >> EXTRA_OECONF = "--disable-documentation --disable-scanner" >> >> +PR = "r3" >> + >> +COMPATIBLE_MACHINE = "dra7xx-evm" > > So, the only thing you do here is set COMPATIBLE_MACHINE and switch from > release tarball to a specific git commit. What's the point of creating a .inc > file, as I don't see it being used outside of this one recipe... > 1. Yes, the COMPATIBLE_MACHINE was incorporated because of the comment from V1 patch. 2. We are using the git versions because it allows us to revert some patches easily during the active development (I mean reverting some already existing commits in the upstream). 3. In the past GLSDK supported 1.0.6 and 1.3.0 version. Both the recipes had a common portion. The reason of creating the .inc file was to include the common portions of the wayland recipe and only have the non common portions in the actual recipe. The inc file could evolve as per the need, but I can rework and submit a single file if you think the inc file would not be necessary. If you look at the difference between 1.3.0 and 1.5.0, there are lots of similarities in the common portion which can go into the inc file. > >> +SRC_URI = "git://anongit.freedesktop.org/wayland/wayland;protocol=git" >> +SRC_URI_append_class-native = " file://just-scanner.patch" >> + >> +S = "${WORKDIR}/git" >> + >> +FILESEXTRAPATHS_prepend := "${THISDIR}:" >> + >> +SRCREV = "2c3dbb89031f120fb191492634e53a583f99d57d" >> + >> # Wayland installs a M4 macro for other projects to use, which uses the target >> # pkg-config to find files. Replace pkg-config with pkg-config-native. >> do_install_append_class-native() { >> @@ -34,3 +31,4 @@ do_install_append_class-native() { >> -e 's,$PKG_CONFIG,pkg-config-native,g' \ >> -i ${D}/${datadir}/aclocal/wayland-scanner.m4 >> } >> + >> -- >> 2.0.1 >> >> _______________________________________________ >> meta-arago mailing list >> meta-arago@arago-project.org >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago