[gentoo-commits] repo/gentoo:master commit in: dev-gap/polycyclic/, dev-gap/polycyclic/files/

[gentoo-commits] repo/gentoo:master commit in: dev-gap/polycyclic/, dev-gap/polycyclic/files/

[Michael Orlitzky]

commit:     4217dbbe01a264196b0b693e4de1ffb306b80d6c
Author:     Michael Orlitzky <mjo <AT> gentoo <DOT> org>
AuthorDate: Tue Mar 19 20:43:39 2024 +0000
Commit:     Michael Orlitzky <mjo <AT> gentoo <DOT> org>
CommitDate: Tue Mar 19 20:45:34 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4217dbbe

dev-gap/polycyclic: patch out a failing test

Upstream has already recommended that we disable the test suite for this
package, so we may be on our own here, but this one is again easy to
patch out and the rest of the test suite passes.

Signed-off-by: Michael Orlitzky <mjo <AT> gentoo.org>

 .../files/polycyclic-2.16-failing-test.patch       | 28 ++++++++++++++++++++++
 dev-gap/polycyclic/polycyclic-2.16.ebuild          |  6 ++++-
 2 files changed, 33 insertions(+), 1 deletion(-)

diff --git a/dev-gap/polycyclic/files/polycyclic-2.16-failing-test.patch b/dev-gap/polycyclic/files/polycyclic-2.16-failing-test.patch
new file mode 100644
index 000000000000..015f5668eba4
--- /dev/null
+++ b/dev-gap/polycyclic/files/polycyclic-2.16-failing-test.patch
@@ -0,0 +1,28 @@
+From 019d573ddbade01304f62b9cd07b4e759e21398a Mon Sep 17 00:00:00 2001
+From: Michael Orlitzky <michael@orlitzky.com>
+Date: Tue, 19 Mar 2024 16:38:13 -0400
+Subject: [PATCH] tst/bugfix.tst: delete failing test
+
+https://github.com/gap-packages/polycyclic/issues/89
+---
+ tst/bugfix.tst | 4 ----
+ 1 file changed, 4 deletions(-)
+
+diff --git a/tst/bugfix.tst b/tst/bugfix.tst
+index 393390e..9293671 100644
+--- a/tst/bugfix.tst
++++ b/tst/bugfix.tst
+@@ -58,10 +58,6 @@ gap> NonAbelianTensorSquare(Image(iso*iso1));
+ Pcp-group with orders [ 2, 2, 3, 2, 2, 2, 2 ]
+ 
+ #
+-gap> # The problem with the previous example is/was that Igs(G)
+-gap> # is set to a non-standard value:
+-gap> Igs(G);
+-[ g1, g2*g5, g3*g4*g5^2, g4*g5, g5 ]
+ gap> # Unfortunately, it seems that a lot of code that
+ gap> # really should be using Ngs or Cgs is using Igs incorrectly.
+ gap> # For example, direct products could return *invalid* embeddings:
+-- 
+2.43.2
+

diff --git a/dev-gap/polycyclic/polycyclic-2.16.ebuild b/dev-gap/polycyclic/polycyclic-2.16.ebuild
index a43edbe76df6..ef301edb14d0 100644
--- a/dev-gap/polycyclic/polycyclic-2.16.ebuild
+++ b/dev-gap/polycyclic/polycyclic-2.16.ebuild
@@ -22,7 +22,11 @@ PDEPEND="dev-gap/alnuth"
 # into them, upstream recommends disabling it:
 #
 #  https://github.com/gap-packages/polycyclic/issues/46
+#  https://github.com/gap-packages/polycyclic/issues/89
 #
-PATCHES=( "${FILESDIR}/${P}-hanging-tests.patch" )
+PATCHES=(
+	"${FILESDIR}/${P}-hanging-tests.patch"
+	"${FILESDIR}/${P}-failing-test.patch"
+)
 
 gap-pkg_enable_tests