* [PATCH 1/3] xf86-video-intel: patch cleanup
@ 2018-06-18 10:10 Ross Burton
2018-06-18 10:10 ` [PATCH 2/3] xf86-video-intel: disable DRI 1, enable DRI 3 Ross Burton
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Ross Burton @ 2018-06-18 10:10 UTC (permalink / raw)
To: openembedded-core
always_include_xorg_server.h.patch is merged upstream, so drop it.
disable-x11-dri3.patch was purely to solve determinism issues with the shared sysroot, which
we don't have anymore.
Also remove redundant tarball checksums as this recipe fetches from git.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
.../always_include_xorg_server.h.patch | 24 ----------------------
.../xf86-video-intel/disable-x11-dri3.patch | 17 ---------------
.../xorg-driver/xf86-video-intel_git.bb | 4 ----
3 files changed, 45 deletions(-)
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch
delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch b/meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch
deleted file mode 100644
index 8a5dd39ae1e..00000000000
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch
+++ /dev/null
@@ -1,24 +0,0 @@
-Include xorg-server.h to fix build errors seen with glibc 2.20
-
-In file included from /home/ubuntu/work/upstream/openembedded-core/build/tmp-glibc/sysroots/qemux86/usr/include/string.h:634:0,
- from /home/ubuntu/work/upstream/openembedded-core/build/tmp-glibc/work/i586-oe-linux/xf86-video-intel/2_2.99.912-r0/xf86-video-intel-2.99.912/src/backlight.c:39:
-/home/ubuntu/work/upstream/openembedded-core/build/tmp-glibc/sysroots/qemux86/usr/include/xorg/os.h:579:1: error: expected identifier or '(' before '__extension__'
- strndup(const char *str, size_t n);
- ^
-
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
-
-Upstream-Status: Pending
-
-Index: xf86-video-intel-2.99.912/src/backlight.c
-===================================================================
---- xf86-video-intel-2.99.912.orig/src/backlight.c 2014-04-09 10:41:18.000000000 -0700
-+++ xf86-video-intel-2.99.912/src/backlight.c 2014-08-28 18:52:25.524070587 -0700
-@@ -29,6 +29,7 @@
- #include "config.h"
- #endif
-
-+#include <xorg-server.h>
- #include <sys/types.h>
- #include <sys/wait.h>
- #include <sys/stat.h>
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch b/meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch
deleted file mode 100644
index 3dddd334bf4..00000000000
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch
+++ /dev/null
@@ -1,17 +0,0 @@
-These modules are only used for tests that don't get installed, and the
-automatic detection without explicit dependencies means that a build can find
-e.g. xshmfence at configure time but then it's been removed from the sysroot by
-the time compile happens.
-
-As the tests don't get installed, just disable the checks.
-
-Upstream-Status: Inappropriate [OE-specific]
-Signed-off-by: Ross Burton <ross.burton@intel.com>
-
-diff --git a/configure.ac b/configure.ac
-index bd654f3..78a0e0d 100644
---- a/configure.ac
-+++ b/configure.ac
-@@ -279 +279 @@ fi
--PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-xfixes xcb-present x11-xcb xshmfence x11 xcomposite xdamage xrender xrandr xxf86vm xext libdrm], [x11_dri3="yes"], [x11_dri3="no"])
-+#PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-xfixes xcb-present x11-xcb xshmfence x11 xcomposite xdamage xrender xrandr xxf86vm xext libdrm], [x11_dri3="yes"], [x11_dri3="no"])
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
index 893f5ce3f64..a60da70840d 100644
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
+++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
@@ -14,13 +14,9 @@ PV = "2.99.917+git${SRCPV}"
S = "${WORKDIR}/git"
SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-intel \
- file://disable-x11-dri3.patch \
- file://always_include_xorg_server.h.patch \
file://0001-Add-Coffeelake-PCI-IDs-for-S-Skus.patch \
"
-SRC_URI[md5sum] = "fa196a66e52c0c624fe5d350af7a5e7b"
-SRC_URI[sha256sum] = "00b781eea055582820a123c47b62411bdf6aabf4f03dc0568faec55faf9667c9"
UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>\d+(\.\d+)+)"
DEPENDS += "virtual/libx11 drm libpciaccess pixman"
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/3] xf86-video-intel: disable DRI 1, enable DRI 3
2018-06-18 10:10 [PATCH 1/3] xf86-video-intel: patch cleanup Ross Burton
@ 2018-06-18 10:10 ` Ross Burton
2018-06-18 10:10 ` [PATCH 3/3] xf86-video-intel: update dependencies for the tools Ross Burton
2018-06-18 14:43 ` [PATCH 1/3] xf86-video-intel: patch cleanup akuster808
2 siblings, 0 replies; 4+ messages in thread
From: Ross Burton @ 2018-06-18 10:10 UTC (permalink / raw)
To: openembedded-core
DRI 1 is only needed for UMS on i810[1], and modern X servers (1.20 onwards)
don't support DRI 1, so disable DRI 1.
However, DRI 3 isn't new and unreliable anymore, so enable support for it.
Also, now that the *proto packages are all part of xorgproto which is a required
DEPENDS, we don't need to mention it over and over.
[1] https://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?id=359477215092ac1b602ad1e2f17a28963d9224c2
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
index a60da70840d..cda1de1eb1a 100644
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
+++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
@@ -21,12 +21,12 @@ UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>\d+(\.\d+)+)"
DEPENDS += "virtual/libx11 drm libpciaccess pixman"
-PACKAGECONFIG ??= "xvmc uxa udev ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'dri dri1 dri2', '', d)}"
+PACKAGECONFIG ??= "xvmc uxa udev ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'dri dri2 dri3', '', d)}"
PACKAGECONFIG[dri] = "--enable-dri,--disable-dri"
-PACKAGECONFIG[dri1] = "--enable-dri1,--disable-dri1,xorgproto"
-PACKAGECONFIG[dri2] = "--enable-dri2,--disable-dri2,xorgproto"
-PACKAGECONFIG[dri3] = "--enable-dri3,--disable-dri3,xorgproto"
+PACKAGECONFIG[dri1] = "--enable-dri1,--disable-dri1"
+PACKAGECONFIG[dri2] = "--enable-dri2,--disable-dri2"
+PACKAGECONFIG[dri3] = "--enable-dri3,--disable-dri3"
PACKAGECONFIG[sna] = "--enable-sna,--disable-sna"
PACKAGECONFIG[uxa] = "--enable-uxa,--disable-uxa"
PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev"
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 3/3] xf86-video-intel: update dependencies for the tools
2018-06-18 10:10 [PATCH 1/3] xf86-video-intel: patch cleanup Ross Burton
2018-06-18 10:10 ` [PATCH 2/3] xf86-video-intel: disable DRI 1, enable DRI 3 Ross Burton
@ 2018-06-18 10:10 ` Ross Burton
2018-06-18 14:43 ` [PATCH 1/3] xf86-video-intel: patch cleanup akuster808
2 siblings, 0 replies; 4+ messages in thread
From: Ross Burton @ 2018-06-18 10:10 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
index cda1de1eb1a..5cc4eefc785 100644
--- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
+++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
@@ -31,7 +31,7 @@ PACKAGECONFIG[sna] = "--enable-sna,--disable-sna"
PACKAGECONFIG[uxa] = "--enable-uxa,--disable-uxa"
PACKAGECONFIG[udev] = "--enable-udev,--disable-udev,udev"
PACKAGECONFIG[xvmc] = "--enable-xvmc,--disable-xvmc,libxvmc xcb-util"
-PACKAGECONFIG[tools] = "--enable-tools,--disable-tools,libxinerama libxrandr libxdamage libxfixes libxcursor libxtst libxext libxrender"
+PACKAGECONFIG[tools] = "--enable-tools,--disable-tools,libxinerama libxrandr libxdamage libxfixes libxcursor libxtst libxrender libxscrnsaver libxext libx11 pixman libxcb libxshmfence"
# --enable-kms-only option is required by ROOTLESS_X
EXTRA_OECONF += '${@oe.utils.conditional( "ROOTLESS_X", "1", " --enable-kms-only", "", d )}'
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/3] xf86-video-intel: patch cleanup
2018-06-18 10:10 [PATCH 1/3] xf86-video-intel: patch cleanup Ross Burton
2018-06-18 10:10 ` [PATCH 2/3] xf86-video-intel: disable DRI 1, enable DRI 3 Ross Burton
2018-06-18 10:10 ` [PATCH 3/3] xf86-video-intel: update dependencies for the tools Ross Burton
@ 2018-06-18 14:43 ` akuster808
2 siblings, 0 replies; 4+ messages in thread
From: akuster808 @ 2018-06-18 14:43 UTC (permalink / raw)
To: Ross Burton, openembedded-core
On 06/18/2018 03:10 AM, Ross Burton wrote:
> always_include_xorg_server.h.patch is merged upstream, so drop it.
>
> disable-x11-dri3.patch was purely to solve determinism issues with the shared sysroot, which
> we don't have anymore.
>
> Also remove redundant tarball checksums as this recipe fetches from git.
cool, I will drop the changes I have queue fro this package.
thanks,
Armin
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
> .../always_include_xorg_server.h.patch | 24 ----------------------
> .../xf86-video-intel/disable-x11-dri3.patch | 17 ---------------
> .../xorg-driver/xf86-video-intel_git.bb | 4 ----
> 3 files changed, 45 deletions(-)
> delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch
> delete mode 100644 meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch
>
> diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch b/meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch
> deleted file mode 100644
> index 8a5dd39ae1e..00000000000
> --- a/meta/recipes-graphics/xorg-driver/xf86-video-intel/always_include_xorg_server.h.patch
> +++ /dev/null
> @@ -1,24 +0,0 @@
> -Include xorg-server.h to fix build errors seen with glibc 2.20
> -
> -In file included from /home/ubuntu/work/upstream/openembedded-core/build/tmp-glibc/sysroots/qemux86/usr/include/string.h:634:0,
> - from /home/ubuntu/work/upstream/openembedded-core/build/tmp-glibc/work/i586-oe-linux/xf86-video-intel/2_2.99.912-r0/xf86-video-intel-2.99.912/src/backlight.c:39:
> -/home/ubuntu/work/upstream/openembedded-core/build/tmp-glibc/sysroots/qemux86/usr/include/xorg/os.h:579:1: error: expected identifier or '(' before '__extension__'
> - strndup(const char *str, size_t n);
> - ^
> -
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -
> -Upstream-Status: Pending
> -
> -Index: xf86-video-intel-2.99.912/src/backlight.c
> -===================================================================
> ---- xf86-video-intel-2.99.912.orig/src/backlight.c 2014-04-09 10:41:18.000000000 -0700
> -+++ xf86-video-intel-2.99.912/src/backlight.c 2014-08-28 18:52:25.524070587 -0700
> -@@ -29,6 +29,7 @@
> - #include "config.h"
> - #endif
> -
> -+#include <xorg-server.h>
> - #include <sys/types.h>
> - #include <sys/wait.h>
> - #include <sys/stat.h>
> diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch b/meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch
> deleted file mode 100644
> index 3dddd334bf4..00000000000
> --- a/meta/recipes-graphics/xorg-driver/xf86-video-intel/disable-x11-dri3.patch
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -These modules are only used for tests that don't get installed, and the
> -automatic detection without explicit dependencies means that a build can find
> -e.g. xshmfence at configure time but then it's been removed from the sysroot by
> -the time compile happens.
> -
> -As the tests don't get installed, just disable the checks.
> -
> -Upstream-Status: Inappropriate [OE-specific]
> -Signed-off-by: Ross Burton <ross.burton@intel.com>
> -
> -diff --git a/configure.ac b/configure.ac
> -index bd654f3..78a0e0d 100644
> ---- a/configure.ac
> -+++ b/configure.ac
> -@@ -279 +279 @@ fi
> --PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-xfixes xcb-present x11-xcb xshmfence x11 xcomposite xdamage xrender xrandr xxf86vm xext libdrm], [x11_dri3="yes"], [x11_dri3="no"])
> -+#PKG_CHECK_MODULES(X11_DRI3, [xcb-dri3 xcb-sync xcb-xfixes xcb-present x11-xcb xshmfence x11 xcomposite xdamage xrender xrandr xxf86vm xext libdrm], [x11_dri3="yes"], [x11_dri3="no"])
> diff --git a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> index 893f5ce3f64..a60da70840d 100644
> --- a/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> +++ b/meta/recipes-graphics/xorg-driver/xf86-video-intel_git.bb
> @@ -14,13 +14,9 @@ PV = "2.99.917+git${SRCPV}"
> S = "${WORKDIR}/git"
>
> SRC_URI = "git://anongit.freedesktop.org/xorg/driver/xf86-video-intel \
> - file://disable-x11-dri3.patch \
> - file://always_include_xorg_server.h.patch \
> file://0001-Add-Coffeelake-PCI-IDs-for-S-Skus.patch \
> "
>
> -SRC_URI[md5sum] = "fa196a66e52c0c624fe5d350af7a5e7b"
> -SRC_URI[sha256sum] = "00b781eea055582820a123c47b62411bdf6aabf4f03dc0568faec55faf9667c9"
> UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>\d+(\.\d+)+)"
>
> DEPENDS += "virtual/libx11 drm libpciaccess pixman"
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-06-18 14:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-18 10:10 [PATCH 1/3] xf86-video-intel: patch cleanup Ross Burton
2018-06-18 10:10 ` [PATCH 2/3] xf86-video-intel: disable DRI 1, enable DRI 3 Ross Burton
2018-06-18 10:10 ` [PATCH 3/3] xf86-video-intel: update dependencies for the tools Ross Burton
2018-06-18 14:43 ` [PATCH 1/3] xf86-video-intel: patch cleanup akuster808
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.