Gentoo Archives: gentoo-commits

From: Sam James <sam@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: media-libs/imlib2/files/, media-libs/imlib2/
Date: Thu, 01 Dec 2022 01:52:45
Message-Id: 1669859533.40383904bf9e4b5c0ee8e55b9587112b7351e7cd.sam@gentoo
1 commit: 40383904bf9e4b5c0ee8e55b9587112b7351e7cd
2 Author: Matoro Mahri <matoro <AT> users <DOT> noreply <DOT> github <DOT> com>
3 AuthorDate: Wed Nov 30 22:20:40 2022 +0000
4 Commit: Sam James <sam <AT> gentoo <DOT> org>
5 CommitDate: Thu Dec 1 01:52:13 2022 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=40383904
7
8 media-libs/imlib2: backport BE build fix to 1.7.5
9
10 No revbump since this is build-only.
11
12 Closes: https://bugs.gentoo.org/877777
13 Signed-off-by: Matoro Mahri <matoro <AT> users.noreply.github.com>
14 Closes: https://github.com/gentoo/gentoo/pull/28488
15 Signed-off-by: Sam James <sam <AT> gentoo.org>
16
17 media-libs/imlib2/files/7d60151ba9.patch | 37 ++++++++++++++++++++++++++++++++
18 media-libs/imlib2/imlib2-1.7.5.ebuild | 1 +
19 2 files changed, 38 insertions(+)
20
21 diff --git a/media-libs/imlib2/files/7d60151ba9.patch b/media-libs/imlib2/files/7d60151ba9.patch
22 new file mode 100644
23 index 000000000000..33709d83849f
24 --- /dev/null
25 +++ b/media-libs/imlib2/files/7d60151ba9.patch
26 @@ -0,0 +1,37 @@
27 +https://bugs.gentoo.org/877777
28 +https://git.enlightenment.org/old/legacy-imlib2/commit/7d60151ba9696ef07be79af68d5c631a97c63906
29 +
30 +From 7d60151ba9696ef07be79af68d5c631a97c63906 Mon Sep 17 00:00:00 2001
31 +From: =?UTF-8?q?S=C3=B6ren=20Tempel?= <soeren+git@×××××××××××××.net>
32 +Date: Mon, 3 Jan 2022 10:56:03 +0100
33 +Subject: [PATCH] ICO loader: Fix compilation on big endian architectures
34 +
35 +Commit ff79901a071a76ec73cc98c7ff15102c514afb7b refactors the
36 +ico_read_idir function and removed the local nr variable. Unfortunately,
37 +this variable is still used within an `#ifdef WORDS_BIGENDIAN` block on
38 +big endian architectures as a for loop index variable. As such, the code
39 +does presently not compile since the aforementioned commit. This patch
40 +fixes this issue by re-introducing the variable conditionally on big
41 +endian architectures.
42 +
43 +Note: It would likely be cleaner to declare the nr variable as part of
44 +the loop declaration, however, this C99 feature does not seem to be used
45 +anywhere in the code base, hence I refrained from using it here.
46 +---
47 + src/modules/loaders/loader_ico.c | 3 +++
48 + 1 file changed, 3 insertions(+)
49 +
50 +diff --git a/src/modules/loaders/loader_ico.c b/src/modules/loaders/loader_ico.c
51 +index e8cef41..66c3643 100644
52 +--- a/src/modules/loaders/loader_ico.c
53 ++++ b/src/modules/loaders/loader_ico.c
54 +@@ -139,6 +139,9 @@ ico_read_icon(ico_t * ico, int ino)
55 + {
56 + ie_t *ie;
57 + unsigned int size;
58 ++#ifdef WORDS_BIGENDIAN
59 ++ unsigned int nr;
60 ++#endif
61 +
62 + ie = &ico->ie[ino];
63 +
64
65 diff --git a/media-libs/imlib2/imlib2-1.7.5.ebuild b/media-libs/imlib2/imlib2-1.7.5.ebuild
66 index 8e11a1e78052..f7ae3856cd6a 100644
67 --- a/media-libs/imlib2/imlib2-1.7.5.ebuild
68 +++ b/media-libs/imlib2/imlib2-1.7.5.ebuild
69 @@ -37,6 +37,7 @@ RDEPEND="
70 DEPEND="${RDEPEND}
71 X? ( x11-base/xorg-proto )"
72 BDEPEND="virtual/pkgconfig"
73 +PATCHES=( "${FILESDIR}/7d60151ba9.patch" )
74
75 multilib_src_configure() {
76 local myeconfargs=(