commit:     cf57ed6790be934bfc32398f3d10865da24b1ab0
Author:     orbea <orbea <AT> riseup <DOT> net>
AuthorDate: Thu Sep 14 19:12:46 2023 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Fri Sep 15 05:57:11 2023 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=cf57ed67

games-emulation/melonds-jg: fix i686 build issue

On a crossdev environment with x86_64-pc-linux-musl host and
i686-pc-linux-musl build a static assert fails in the vendored teakra
test_generator.cpp, but teakra was modified by the melonds upstream
enough that a system version is not possible.

However the jollygood port doesn't actually use any of the code in
test_generator.cpp so it can be simply removed from the build. As this
is unused code a revdump is not required.

Closes: https://bugs.gentoo.org/914196
Upstream-PR: https://gitlab.com/jgemu/melonds/-/merge_requests/9
Upstream-Commit: 
https://gitlab.com/jgemu/melonds/-/commit/0953b8874187bfdd35a0bf787c3bdbb6cc65dc9d
Signed-off-by: orbea <orbea <AT> riseup.net>
Closes: https://github.com/gentoo/gentoo/pull/32784
Signed-off-by: Sam James <sam <AT> gentoo.org>

 ...melonds-jg-0.9.5-teakra-no-test_generator.patch | 38 ++++++++++++++++++++++
 games-emulation/melonds-jg/melonds-jg-0.9.5.ebuild |  1 +
 2 files changed, 39 insertions(+)

diff --git 
a/games-emulation/melonds-jg/files/melonds-jg-0.9.5-teakra-no-test_generator.patch
 
b/games-emulation/melonds-jg/files/melonds-jg-0.9.5-teakra-no-test_generator.patch
new file mode 100644
index 000000000000..a482e93a52fe
--- /dev/null
+++ 
b/games-emulation/melonds-jg/files/melonds-jg-0.9.5-teakra-no-test_generator.patch
@@ -0,0 +1,38 @@
+https://bugs.gentoo.org/914196
+https://gitlab.com/jgemu/melonds/-/merge_requests/9
+https://gitlab.com/jgemu/melonds/-/commit/0953b8874187bfdd35a0bf787c3bdbb6cc65dc9d
+
+From 0953b8874187bfdd35a0bf787c3bdbb6cc65dc9d Mon Sep 17 00:00:00 2001
+From: orbea <[email protected]>
+Date: Thu, 14 Sep 2023 09:58:18 -0700
+Subject: [PATCH] jollygood: don't build teakra's test_generator.cpp
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+This is unused in the jollygood core and causes build failures for i686.
+
+As reproduced on Gentoo:
+
+In file included from 
/usr/i686-pc-linux-musl/tmp/portage/games-emulation/melonds-jg-0.9.5/work/melonds-0.9.5/jollygood/../src/teakra/src/test_generator.cpp:10:
+/usr/i686-pc-linux-musl/tmp/portage/games-emulation/melonds-jg-0.9.5/work/melonds-0.9.5/jollygood/../src/teakra/src/test.h:37:32:
 error: static assertion failed
+   37 | static_assert(sizeof(TestCase) == 4312);
+      |               ~~~~~~~~~~~~~~~~~^~~~~~~
+/usr/i686-pc-linux-musl/tmp/portage/games-emulation/melonds-jg-0.9.5/work/melonds-0.9.5/jollygood/../src/teakra/src/test.h:37:32:
 note: the comparison reduces to ‘(4308 == 4312)’
+make: *** [Makefile:169: objs/src/teakra/src/test_generator.o] Error 1
+---
+ jollygood/Makefile | 1 -
+ 1 file changed, 1 deletion(-)
+
+diff --git a/jollygood/Makefile b/jollygood/Makefile
+index 03f34a1..c206f05 100644
+--- a/jollygood/Makefile
++++ b/jollygood/Makefile
+@@ -68,7 +68,6 @@ CXXSRCS := src/teakra/src/ahbm.cpp \
+       src/teakra/src/parser.cpp \
+       src/teakra/src/processor.cpp \
+       src/teakra/src/teakra.cpp \
+-      src/teakra/src/test_generator.cpp \
+       src/teakra/src/timer.cpp \
+       src/ARCodeFile.cpp \
+       src/AREngine.cpp \

diff --git a/games-emulation/melonds-jg/melonds-jg-0.9.5.ebuild 
b/games-emulation/melonds-jg/melonds-jg-0.9.5.ebuild
index 56734daea220..b7c9f13cf065 100644
--- a/games-emulation/melonds-jg/melonds-jg-0.9.5.ebuild
+++ b/games-emulation/melonds-jg/melonds-jg-0.9.5.ebuild
@@ -35,6 +35,7 @@ BDEPEND="
 
 PATCHES=(
        "${FILESDIR}"/${P}-format.patch
+       "${FILESDIR}"/${P}-teakra-no-test_generator.patch # 914196
 )
 
 src_compile() {

Reply via email to