All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-core@lists.openembedded.org,
	Alexander Kanavin <alexander.kanavin@linux.intel.com>
Cc: openembedded-commits@lists.openembedded.org
Subject: Re: [oe-commits] [openembedded-core] 82/103: libdrm: update to 2.4.67
Date: Tue, 26 Apr 2016 16:38:40 +0200	[thread overview]
Message-ID: <20160426143840.GD2559@jama> (raw)
In-Reply-To: <20160228113520.997E9501B9@opal.openembedded.org>

[-- Attachment #1: Type: text/plain, Size: 8887 bytes --]

On Sun, Feb 28, 2016 at 11:36:22AM +0000, git@git.openembedded.org wrote:
> rpurdie pushed a commit to branch master
> in repository openembedded-core.
> 
> commit be2f3e78742cfd94d1d1ad1f67bb651b12adf357
> Author: Alexander Kanavin <alexander.kanavin@linux.intel.com>
> AuthorDate: Wed Feb 24 17:13:22 2016 +0200
> 
>     libdrm: update to 2.4.67
>     
>     Drop 0001-tests-kms-steal-crtc-Include-sys-select.h.patch and detect.patch
>     (merged upstream).

Something is wrong with the cunit patch in upstream, now it has 2 autodetected
dependencies.
 
WARN: libdrm: libdrm-tests rdepends on cunit, but it isn't a build dependency?
WARN: libdrm: libdrm-tests rdepends on libdrm-amdgpu, but it isn't a build dependency?

>     
>     Signed-off-by: Alexander Kanavin <alexander.kanavin@linux.intel.com>
>     Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  ...tests-kms-steal-crtc-Include-sys-select.h.patch | 96 ----------------------
>  meta/recipes-graphics/drm/libdrm/detect.patch      | 59 -------------
>  .../drm/{libdrm_2.4.66.bb => libdrm_2.4.67.bb}     |  6 +-
>  3 files changed, 2 insertions(+), 159 deletions(-)
> 
> diff --git a/meta/recipes-graphics/drm/libdrm/0001-tests-kms-steal-crtc-Include-sys-select.h.patch b/meta/recipes-graphics/drm/libdrm/0001-tests-kms-steal-crtc-Include-sys-select.h.patch
> deleted file mode 100644
> index e4ecbac..0000000
> --- a/meta/recipes-graphics/drm/libdrm/0001-tests-kms-steal-crtc-Include-sys-select.h.patch
> +++ /dev/null
> @@ -1,96 +0,0 @@
> -From fb53d7a1be116a172b56625f290fcf88b6d158ca Mon Sep 17 00:00:00 2001
> -From: Khem Raj <raj.khem@gmail.com>
> -Date: Wed, 20 Jan 2016 05:23:00 +0000
> -Subject: [PATCH] tests: Include sys/select.h
> -
> -Used in compliance with POSIX 2001/2008
> -
> -Fixes errors e.g.
> -error: implicit declaration of function 'select'
> -
> -and helps with missing definitions of FD_* defines
> -
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -Reviewed-by: Thierry Reding <thierry.reding@gmail.com>
> ----
> -Upstream-Status: Submitted
> -
> - configure.ac                     | 2 +-
> - tests/kms/kms-steal-crtc.c       | 3 +++
> - tests/kms/kms-universal-planes.c | 3 +++
> - tests/modetest/modetest.c        | 3 +++
> - tests/vbltest/vbltest.c          | 3 +++
> - 5 files changed, 13 insertions(+), 1 deletion(-)
> -
> -diff --git a/configure.ac b/configure.ac
> -index 057a846..258e2f6 100644
> ---- a/configure.ac
> -+++ b/configure.ac
> -@@ -53,7 +53,7 @@ AC_USE_SYSTEM_EXTENSIONS
> - AC_SYS_LARGEFILE
> - AC_FUNC_ALLOCA
> - 
> --AC_CHECK_HEADERS([sys/mkdev.h sys/sysctl.h])
> -+AC_CHECK_HEADERS([sys/mkdev.h sys/sysctl.h sys/select.h])
> - 
> - # Initialize libtool
> - LT_PREREQ([2.2])
> -diff --git a/tests/kms/kms-steal-crtc.c b/tests/kms/kms-steal-crtc.c
> -index 2f7f327..497772e 100644
> ---- a/tests/kms/kms-steal-crtc.c
> -+++ b/tests/kms/kms-steal-crtc.c
> -@@ -31,6 +31,9 @@
> - #include <stdio.h>
> - #include <string.h>
> - #include <unistd.h>
> -+#ifdef HAVE_SYS_SELECT_H
> -+#include <sys/select.h>
> -+#endif
> - 
> - #include <drm_fourcc.h>
> - 
> -diff --git a/tests/kms/kms-universal-planes.c b/tests/kms/kms-universal-planes.c
> -index 9151231..d8e5fc4 100644
> ---- a/tests/kms/kms-universal-planes.c
> -+++ b/tests/kms/kms-universal-planes.c
> -@@ -32,6 +32,9 @@
> - #include <stdio.h>
> - #include <string.h>
> - #include <unistd.h>
> -+#ifdef HAVE_SYS_SELECT_H
> -+#include <sys/select.h>
> -+#endif
> - 
> - #include <drm_fourcc.h>
> - #include "xf86drm.h"
> -diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
> -index 22e3e81..d5a4062 100644
> ---- a/tests/modetest/modetest.c
> -+++ b/tests/modetest/modetest.c
> -@@ -55,6 +55,9 @@
> - #include <errno.h>
> - #include <sys/poll.h>
> - #include <sys/time.h>
> -+#ifdef HAVE_SYS_SELECT_H
> -+#include <sys/select.h>
> -+#endif
> - 
> - #include "xf86drm.h"
> - #include "xf86drmMode.h"
> -diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c
> -index 1833321..086407e 100644
> ---- a/tests/vbltest/vbltest.c
> -+++ b/tests/vbltest/vbltest.c
> -@@ -37,6 +37,9 @@
> - #include <errno.h>
> - #include <sys/poll.h>
> - #include <sys/time.h>
> -+#ifdef HAVE_SYS_SELECT_H
> -+#include <sys/select.h>
> -+#endif
> - 
> - #include "xf86drm.h"
> - #include "xf86drmMode.h"
> --- 
> -2.7.0
> -
> diff --git a/meta/recipes-graphics/drm/libdrm/detect.patch b/meta/recipes-graphics/drm/libdrm/detect.patch
> deleted file mode 100644
> index 6397a59..0000000
> --- a/meta/recipes-graphics/drm/libdrm/detect.patch
> +++ /dev/null
> @@ -1,59 +0,0 @@
> -Upstream-Status: Backport
> -Signed-off-by: Ross Burton <ross.burton@intel.com>
> -
> -commit 798022b61c58d945f9027c823a188dcedecd3d06
> -Author: Marcin Ślusarz <marcin.slusarz@gmail.com>
> -Date:   Sun Jan 24 13:17:34 2016 +0100
> -
> -    configure.ac: don't detect disabled options dependencies
> -    
> -    Currently with --disable-amdgpu --disable-valgrind --disable-cairo-tests
> -    cunit, valgrind and cairo are still detected.
> -    
> -    Signed-off-by: Marcin Ślusarz <marcin.slusarz@gmail.com>
> -    Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
> -
> -diff --git a/configure.ac b/configure.ac
> -index 057a846..a09be61 100644
> ---- a/configure.ac
> -+++ b/configure.ac
> -@@ -363,12 +363,14 @@ fi
> --# Detect cunit library
> --PKG_CHECK_MODULES([CUNIT], [cunit >= 2.1], [have_cunit=yes], [have_cunit=no])
> --# If pkg-config does not find cunit, check it using AC_CHECK_LIB.  We
> --# do this because Debian (Ubuntu) lacks pkg-config file for cunit.
> --# fixed in 2.1-2.dfsg-3: http://anonscm.debian.org/cgit/collab-maint/cunit.git/commit/?h=debian
> --if test "x${have_cunit}" = "xno"; then
> --	AC_CHECK_LIB([cunit], [CU_initialize_registry], [have_cunit=yes], [have_cunit=no])
> --	if test "x${have_cunit}" = "xyes"; then
> --		CUNIT_LIBS="-lcunit"
> --		CUNIT_CFLAGS=""
> --		AC_SUBST([CUNIT_LIBS])
> --		AC_SUBST([CUNIT_CFLAGS])
> -+if test "x$AMDGPU" != xno; then
> -+	# Detect cunit library
> -+	PKG_CHECK_MODULES([CUNIT], [cunit >= 2.1], [have_cunit=yes], [have_cunit=no])
> -+	# If pkg-config does not find cunit, check it using AC_CHECK_LIB.  We
> -+	# do this because Debian (Ubuntu) lacks pkg-config file for cunit.
> -+	# fixed in 2.1-2.dfsg-3: http://anonscm.debian.org/cgit/collab-maint/cunit.git/commit/?h=debian
> -+	if test "x${have_cunit}" = "xno"; then
> -+		AC_CHECK_LIB([cunit], [CU_initialize_registry], [have_cunit=yes], [have_cunit=no])
> -+		if test "x${have_cunit}" = "xyes"; then
> -+			CUNIT_LIBS="-lcunit"
> -+			CUNIT_CFLAGS=""
> -+			AC_SUBST([CUNIT_LIBS])
> -+			AC_SUBST([CUNIT_CFLAGS])
> -+		fi
> -@@ -375,0 +378,2 @@ if test "x${have_cunit}" = "xno"; then
> -+else
> -+	have_cunit=no
> -@@ -404 +408,3 @@ AC_ARG_ENABLE([cairo-tests],
> --PKG_CHECK_MODULES(CAIRO, cairo, [HAVE_CAIRO=yes], [HAVE_CAIRO=no])
> -+if test "x$CAIRO" != xno; then
> -+	PKG_CHECK_MODULES(CAIRO, cairo, [HAVE_CAIRO=yes], [HAVE_CAIRO=no])
> -+fi
> -@@ -449 +455,3 @@ AC_ARG_ENABLE(valgrind,
> --PKG_CHECK_MODULES(VALGRIND, [valgrind], [have_valgrind=yes], [have_valgrind=no])
> -+if test "x$VALGRIND" != xno; then
> -+	PKG_CHECK_MODULES(VALGRIND, [valgrind], [have_valgrind=yes], [have_valgrind=no])
> -+fi
> diff --git a/meta/recipes-graphics/drm/libdrm_2.4.66.bb b/meta/recipes-graphics/drm/libdrm_2.4.67.bb
> similarity index 87%
> rename from meta/recipes-graphics/drm/libdrm_2.4.66.bb
> rename to meta/recipes-graphics/drm/libdrm_2.4.67.bb
> index 79cd94a..ddd697f 100644
> --- a/meta/recipes-graphics/drm/libdrm_2.4.66.bb
> +++ b/meta/recipes-graphics/drm/libdrm_2.4.67.bb
> @@ -12,13 +12,11 @@ DEPENDS = "libpthread-stubs udev libpciaccess"
>  
>  SRC_URI = "http://dri.freedesktop.org/libdrm/${BP}.tar.bz2 \
>             file://installtests.patch \
> -           file://0001-tests-kms-steal-crtc-Include-sys-select.h.patch \
> -           file://detect.patch \
>             file://fix_O_CLOEXEC_undeclared.patch \
>            "
>  
> -SRC_URI[md5sum] = "c6809c48538d6e5999588832045ff014"
> -SRC_URI[sha256sum] = "79cb8e988749794edfb2d777b298d5292eff353bbbb71ed813589e61d2bc2d76"
> +SRC_URI[md5sum] = "b844c6af34e613ebf52f80b3cd8d7055"
> +SRC_URI[sha256sum] = "ee5b71e1113be37544d0752681c12f040c01f782e2933df7d7bc21fd0d10cebe"
>  
>  inherit autotools pkgconfig
>  
> 
> -- 
> To stop receiving notification emails like this one, please contact
> the administrator of this repository.
> -- 
> _______________________________________________
> Openembedded-commits mailing list
> Openembedded-commits@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-commits

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

           reply	other threads:[~2016-04-26 14:38 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20160228113520.997E9501B9@opal.openembedded.org>]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160426143840.GD2559@jama \
    --to=martin.jansa@gmail.com \
    --cc=alexander.kanavin@linux.intel.com \
    --cc=openembedded-commits@lists.openembedded.org \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.