mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Christian Melki <christian.melki@t2data.com>
To: ptxdist@pengutronix.de
Subject: [ptxdist] [PATCH] wayland-utils: Version bump. 1.0.0 -> 1.1.0
Date: Fri, 18 Nov 2022 21:13:01 +0100	[thread overview]
Message-ID: <20221118201301.1496741-1-christian.melki@t2data.com> (raw)

Some bugfixes and enhancement of this rather new tool.

* Add patch to make it build without libdrm.
Adding libdrm enhances output, but is not needed.
In reality, libdrm is most likely going to be there.

* Move URL to gitlab one. This release isn't hosted on
the old URL yet (any more?).

Signed-off-by: Christian Melki <christian.melki@t2data.com>
---
 ...ayland-info-Fix-build-without-libdrm.patch | 38 +++++++++++++++++++
 patches/wayland-utils-1.1.0/series            |  4 ++
 rules/wayland-utils.make                      |  6 +--
 3 files changed, 45 insertions(+), 3 deletions(-)
 create mode 100644 patches/wayland-utils-1.1.0/0001-wayland-info-Fix-build-without-libdrm.patch
 create mode 100644 patches/wayland-utils-1.1.0/series

diff --git a/patches/wayland-utils-1.1.0/0001-wayland-info-Fix-build-without-libdrm.patch b/patches/wayland-utils-1.1.0/0001-wayland-info-Fix-build-without-libdrm.patch
new file mode 100644
index 000000000..2b994421a
--- /dev/null
+++ b/patches/wayland-utils-1.1.0/0001-wayland-info-Fix-build-without-libdrm.patch
@@ -0,0 +1,38 @@
+From: Olivier Fourdan <ofourdan@redhat.com>
+Date: Wed, 14 Sep 2022 09:07:10 +0200
+Subject: [PATCH] wayland-info: Fix build without libdrm
+
+wayland-info can optionally use libdrm to provide a description of the
+dmabuf format modifiers.
+
+When not using libdrm however, the build fails because "dev_t" is not
+defined.
+
+The definition of "dev_t" comes from <sys/types.h> which is included
+from <libdrm.h>, which is not included without libdrm support, hence the
+build failure.
+
+Simply include <sys/types.h> unconditionally to make sure "dev_t" is
+defined regardless of libdrm support, to fix the build failure.
+
+Closes: https://gitlab.freedesktop.org/wayland/wayland-utils/-/issues/6
+Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
+Fixes: 240cb739 - "Add support for linux_dmabuf version 4"
+Reviewed-by: Simon Ser <contact@emersion.fr>
+Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.com>
+---
+ wayland-info/wayland-info.c | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/wayland-info/wayland-info.c b/wayland-info/wayland-info.c
+index 53cd04bcb95e..98ff2052add7 100644
+--- a/wayland-info/wayland-info.c
++++ b/wayland-info/wayland-info.c
+@@ -34,6 +34,7 @@
+ #include <ctype.h>
+ #include <unistd.h>
+ #include <sys/mman.h>
++#include <sys/types.h>
+ 
+ #include <wayland-client.h>
+ #if HAVE_HUMAN_FORMAT_MODIFIER
diff --git a/patches/wayland-utils-1.1.0/series b/patches/wayland-utils-1.1.0/series
new file mode 100644
index 000000000..21f38d9d7
--- /dev/null
+++ b/patches/wayland-utils-1.1.0/series
@@ -0,0 +1,4 @@
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-wayland-info-Fix-build-without-libdrm.patch
+# bcd03cd405a631566145d1b0ddee2dc4  - git-ptx-patches magic
diff --git a/rules/wayland-utils.make b/rules/wayland-utils.make
index 91942385b..28c2f493b 100644
--- a/rules/wayland-utils.make
+++ b/rules/wayland-utils.make
@@ -14,11 +14,11 @@ PACKAGES-$(PTXCONF_WAYLAND_UTILS) += wayland-utils
 #
 # Paths and names
 #
-WAYLAND_UTILS_VERSION		:= 1.0.0
-WAYLAND_UTILS_MD5		:= 714875aefb10e7f683cde24e58d033ad
+WAYLAND_UTILS_VERSION		:= 1.1.0
+WAYLAND_UTILS_MD5		:= 1a398cbb75c3ac1ac231b2a8f84bd6d6
 WAYLAND_UTILS			:= wayland-utils-$(WAYLAND_UTILS_VERSION)
 WAYLAND_UTILS_SUFFIX		:= tar.xz
-WAYLAND_UTILS_URL		:= https://wayland.freedesktop.org/releases//$(WAYLAND_UTILS).$(WAYLAND_UTILS_SUFFIX)
+WAYLAND_UTILS_URL		:= https://gitlab.freedesktop.org/wayland/wayland-utils/-/releases/$(WAYLAND_UTILS_VERSION)/downloads/$(WAYLAND_UTILS).$(WAYLAND_UTILS_SUFFIX)
 WAYLAND_UTILS_SOURCE		:= $(SRCDIR)/$(WAYLAND_UTILS).$(WAYLAND_UTILS_SUFFIX)
 WAYLAND_UTILS_DIR		:= $(BUILDDIR)/$(WAYLAND_UTILS)
 WAYLAND_UTILS_LICENSE		:= MIT
-- 
2.34.1




             reply	other threads:[~2022-11-18 20:13 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-18 20:13 Christian Melki [this message]
2022-11-22 13:20 ` [ptxdist] [APPLIED] " Michael Olbrich

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=20221118201301.1496741-1-christian.melki@t2data.com \
    --to=christian.melki@t2data.com \
    --cc=ptxdist@pengutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox