summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatoro Mahri <matoro@users.noreply.github.com>2022-11-30 17:20:40 -0500
committerSam James <sam@gentoo.org>2022-12-01 01:52:13 +0000
commit40383904bf9e4b5c0ee8e55b9587112b7351e7cd (patch)
tree2007b57074be5bcece60a3e0da478ab6050352ca
parentapp-arch/bzip2: Support app-alternatives/bzip2 (diff)
downloadgentoo-40383904bf9e4b5c0ee8e55b9587112b7351e7cd.tar.gz
gentoo-40383904bf9e4b5c0ee8e55b9587112b7351e7cd.tar.bz2
gentoo-40383904bf9e4b5c0ee8e55b9587112b7351e7cd.zip
media-libs/imlib2: backport BE build fix to 1.7.5
No revbump since this is build-only. Closes: https://bugs.gentoo.org/877777 Signed-off-by: Matoro Mahri <matoro@users.noreply.github.com> Closes: https://github.com/gentoo/gentoo/pull/28488 Signed-off-by: Sam James <sam@gentoo.org>
-rw-r--r--media-libs/imlib2/files/7d60151ba9.patch37
-rw-r--r--media-libs/imlib2/imlib2-1.7.5.ebuild1
2 files changed, 38 insertions, 0 deletions
diff --git a/media-libs/imlib2/files/7d60151ba9.patch b/media-libs/imlib2/files/7d60151ba9.patch
new file mode 100644
index 000000000000..33709d83849f
--- /dev/null
+++ b/media-libs/imlib2/files/7d60151ba9.patch
@@ -0,0 +1,37 @@
+https://bugs.gentoo.org/877777
+https://git.enlightenment.org/old/legacy-imlib2/commit/7d60151ba9696ef07be79af68d5c631a97c63906
+
+From 7d60151ba9696ef07be79af68d5c631a97c63906 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?S=C3=B6ren=20Tempel?= <soeren+git@soeren-tempel.net>
+Date: Mon, 3 Jan 2022 10:56:03 +0100
+Subject: [PATCH] ICO loader: Fix compilation on big endian architectures
+
+Commit ff79901a071a76ec73cc98c7ff15102c514afb7b refactors the
+ico_read_idir function and removed the local nr variable. Unfortunately,
+this variable is still used within an `#ifdef WORDS_BIGENDIAN` block on
+big endian architectures as a for loop index variable. As such, the code
+does presently not compile since the aforementioned commit. This patch
+fixes this issue by re-introducing the variable conditionally on big
+endian architectures.
+
+Note: It would likely be cleaner to declare the nr variable as part of
+the loop declaration, however, this C99 feature does not seem to be used
+anywhere in the code base, hence I refrained from using it here.
+---
+ src/modules/loaders/loader_ico.c | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/src/modules/loaders/loader_ico.c b/src/modules/loaders/loader_ico.c
+index e8cef41..66c3643 100644
+--- a/src/modules/loaders/loader_ico.c
++++ b/src/modules/loaders/loader_ico.c
+@@ -139,6 +139,9 @@ ico_read_icon(ico_t * ico, int ino)
+ {
+ ie_t *ie;
+ unsigned int size;
++#ifdef WORDS_BIGENDIAN
++ unsigned int nr;
++#endif
+
+ ie = &ico->ie[ino];
+
diff --git a/media-libs/imlib2/imlib2-1.7.5.ebuild b/media-libs/imlib2/imlib2-1.7.5.ebuild
index 8e11a1e78052..f7ae3856cd6a 100644
--- a/media-libs/imlib2/imlib2-1.7.5.ebuild
+++ b/media-libs/imlib2/imlib2-1.7.5.ebuild
@@ -37,6 +37,7 @@ RDEPEND="
DEPEND="${RDEPEND}
X? ( x11-base/xorg-proto )"
BDEPEND="virtual/pkgconfig"
+PATCHES=( "${FILESDIR}/7d60151ba9.patch" )
multilib_src_configure() {
local myeconfargs=(