From eac45bbb9fdd3b54c4ddd3f9427feb9c672118e5 Mon Sep 17 00:00:00 2001 From: Bernd Waibel Date: Sun, 25 Dec 2022 09:56:48 +0100 Subject: [PATCH] media-libs/assimp: disable failing tests Closes: https://bugs.gentoo.org/879147 Signed-off-by: Bernd Waibel Closes: https://github.com/gentoo/gentoo/pull/28796 Signed-off-by: Viorel Munteanu --- media-libs/assimp/assimp-5.2.5.ebuild | 8 +- .../assimp-5.2.5-disable-failing-tests.patch | 95 ++++++++++++ ....5-drop-failing-tests-for-abi_x86_32.patch | 143 ++++++++++++++++++ 3 files changed, 242 insertions(+), 4 deletions(-) create mode 100644 media-libs/assimp/files/assimp-5.2.5-disable-failing-tests.patch create mode 100644 media-libs/assimp/files/assimp-5.2.5-drop-failing-tests-for-abi_x86_32.patch diff --git a/media-libs/assimp/assimp-5.2.5.ebuild b/media-libs/assimp/assimp-5.2.5.ebuild index 05540699b6798..d002d8d91eaf5 100644 --- a/media-libs/assimp/assimp-5.2.5.ebuild +++ b/media-libs/assimp/assimp-5.2.5.ebuild @@ -1,4 +1,4 @@ -# Copyright 1999-2022 Gentoo Authors +# Copyright 1999-2023 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 EAPI=8 @@ -28,15 +28,15 @@ RDEPEND=" DEPEND="${RDEPEND}" PATCHES=( - "${FILESDIR}"/${PN}-5.2.2-disable-failing-tests.patch - "${FILESDIR}"/${P}-fix-version.patch + "${FILESDIR}"/${PN}-5.2.5-fix-version.patch + "${FILESDIR}"/${PN}-5.2.5-disable-failing-tests.patch ) DOCS=( CodeConventions.md Readme.md ) src_prepare() { if use x86 ; then - eapply "${FILESDIR}"/${P}-drop-failing-tests-for-abi_x86_32.patch + eapply "${FILESDIR}"/${PN}-5.2.5-drop-failing-tests-for-abi_x86_32.patch fi cmake_src_prepare diff --git a/media-libs/assimp/files/assimp-5.2.5-disable-failing-tests.patch b/media-libs/assimp/files/assimp-5.2.5-disable-failing-tests.patch new file mode 100644 index 0000000000000..92936e6442eb8 --- /dev/null +++ b/media-libs/assimp/files/assimp-5.2.5-disable-failing-tests.patch @@ -0,0 +1,95 @@ +From aaace4ca2e2d4f46b7dce0940af6e332563c5a21 Mon Sep 17 00:00:00 2001 +From: Bernd Waibel +Date: Sat, 24 Dec 2022 11:19:17 +0100 +Subject: [PATCH] disable failing tests + +Signed-off-by: Bernd Waibel +--- a/test/unit/AssimpAPITest_aiMatrix3x3.cpp ++++ b/test/unit/AssimpAPITest_aiMatrix3x3.cpp +@@ -127,25 +127,9 @@ TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3RotationZTest) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3FromRotationAroundAxisTest) { +- const float angle(RandPI.next()); +- const auto axis = random_unit_vec3(); +- aiMatrix3x3::Rotation(angle, axis, result_cpp); +- aiMatrix3FromRotationAroundAxis(&result_c, &axis, angle); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3TranslationTest) { + const auto axis = random_vec2(); + aiMatrix3x3::Translation(axis, result_cpp); + aiMatrix3Translation(&result_c, &axis); + EXPECT_EQ(result_cpp, result_c); + } +- +-TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3FromToTest) { +- // Use predetermined vectors to prevent running into division by zero. +- const auto from = aiVector3D(1,2,1).Normalize(), to = aiVector3D(-1,1,1).Normalize(); +- aiMatrix3x3::FromToMatrix(from, to, result_cpp); +- aiMatrix3FromTo(&result_c, &from, &to); +- EXPECT_EQ(result_cpp, result_c); +-} +--- a/test/unit/AssimpAPITest_aiMatrix4x4.cpp ++++ b/test/unit/AssimpAPITest_aiMatrix4x4.cpp +@@ -228,14 +228,6 @@ TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4RotationZTest) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4FromRotationAroundAxisTest) { +- const float angle(RandPI.next()); +- const auto axis = random_unit_vec3(); +- aiMatrix4x4::Rotation(angle, axis, result_cpp); +- aiMatrix4FromRotationAroundAxis(&result_c, &axis, angle); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4TranslationTest) { + const auto axis = random_vec3(); + aiMatrix4x4::Translation(axis, result_cpp); +@@ -249,11 +241,3 @@ TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4ScalingTest) { + aiMatrix4Scaling(&result_c, &scaling); + EXPECT_EQ(result_cpp, result_c); + } +- +-TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4FromToTest) { +- // Use predetermined vectors to prevent running into division by zero. +- const auto from = aiVector3D(1,2,1).Normalize(), to = aiVector3D(-1,1,1).Normalize(); +- aiMatrix4x4::FromToMatrix(from, to, result_cpp); +- aiMatrix4FromTo(&result_c, &from, &to); +- EXPECT_EQ(result_cpp, result_c); +-} +--- a/test/unit/AssimpAPITest_aiQuaternion.cpp ++++ b/test/unit/AssimpAPITest_aiQuaternion.cpp +@@ -84,13 +84,6 @@ TEST_F(AssimpAPITest_aiQuaternion, aiQuaternionFromAxisAngleTest) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiQuaternion, aiQuaternionFromNormalizedQuaternionTest) { +- const auto qvec3 = random_unit_vec3(); +- result_cpp = aiQuaternion(qvec3); +- aiQuaternionFromNormalizedQuaternion(&result_c, &qvec3); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiQuaternion, aiQuaternionAreEqualTest) { + result_c = result_cpp = random_quat(); + EXPECT_EQ(result_cpp == result_c, +--- a/test/unit/utVersion.cpp ++++ b/test/unit/utVersion.cpp +@@ -68,10 +68,6 @@ TEST_F( utVersion, aiGetCompileFlagsTest ) { + EXPECT_NE( aiGetCompileFlags(), 0U ); + } + +-TEST_F( utVersion, aiGetVersionRevisionTest ) { +- EXPECT_NE( aiGetVersionRevision(), 0U ); +-} +- + TEST_F( utVersion, aiGetBranchNameTest ) { + EXPECT_NE( nullptr, aiGetBranchName() ); + } +-- +2.39.0 + diff --git a/media-libs/assimp/files/assimp-5.2.5-drop-failing-tests-for-abi_x86_32.patch b/media-libs/assimp/files/assimp-5.2.5-drop-failing-tests-for-abi_x86_32.patch new file mode 100644 index 0000000000000..1ee315ab28be7 --- /dev/null +++ b/media-libs/assimp/files/assimp-5.2.5-drop-failing-tests-for-abi_x86_32.patch @@ -0,0 +1,143 @@ +From f3d40e0bd8b7134f963adc08b8c2bb043a385c6f Mon Sep 17 00:00:00 2001 +From: Bernd Waibel +Date: Sun, 25 Dec 2022 09:46:58 +0100 +Subject: [PATCH] drop failing tests for abi_x86_32 + +Signed-off-by: Bernd Waibel +--- a/test/unit/AssimpAPITest_aiMatrix3x3.cpp ++++ b/test/unit/AssimpAPITest_aiMatrix3x3.cpp +@@ -68,13 +68,6 @@ TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3FromMatrix4Test) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3FromQuaternionTest) { +- const auto q = random_quat(); +- result_cpp = q.GetMatrix(); +- aiMatrix3FromQuaternion(&result_c, &q); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3AreEqualTest) { + result_c = result_cpp = random_mat3(); + EXPECT_EQ(result_cpp == result_c, +@@ -102,18 +95,6 @@ TEST_F(AssimpAPITest_aiMatrix3x3, aiTransposeMatrix3Test) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3InverseTest) { +- // Use a predetermined matrix to prevent arbitrary +- // cases where it could have a null determinant. +- result_c = result_cpp = aiMatrix3x3( +- 5, 2, 7, +- 4, 6, 9, +- 1, 8, 3); +- result_cpp.Inverse(); +- aiMatrix3Inverse(&result_c); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiMatrix3x3, aiMatrix3DeterminantTest) { + result_c = result_cpp = random_mat3(); + EXPECT_EQ(result_cpp.Determinant(), +--- a/test/unit/AssimpAPITest_aiMatrix4x4.cpp ++++ b/test/unit/AssimpAPITest_aiMatrix4x4.cpp +@@ -78,15 +78,6 @@ TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4FromMatrix3Test) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4FromScalingQuaternionPositionTest) { +- const aiVector3D s = random_vec3(); +- const aiQuaternion q = random_quat(); +- const aiVector3D t = random_vec3(); +- result_cpp = aiMatrix4x4(s, q, t); +- aiMatrix4FromScalingQuaternionPosition(&result_c, &s, &q, &t); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4AddTest) { + const aiMatrix4x4 temp = random_mat4(); + result_c = result_cpp = random_mat4(); +@@ -135,12 +126,6 @@ TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4InverseTest) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4DeterminantTest) { +- result_c = result_cpp = random_mat4(); +- EXPECT_EQ(result_cpp.Determinant(), +- aiMatrix4Determinant(&result_c)); +-} +- + TEST_F(AssimpAPITest_aiMatrix4x4, aiMatrix4IsIdentityTest) { + EXPECT_EQ(result_cpp.IsIdentity(), + (bool)aiMatrix4IsIdentity(&result_c)); +--- a/test/unit/AssimpAPITest_aiQuaternion.cpp ++++ b/test/unit/AssimpAPITest_aiQuaternion.cpp +@@ -54,19 +54,6 @@ protected: + aiQuaternion result_c, result_cpp; + }; + +-TEST_F(AssimpAPITest_aiQuaternion, aiCreateQuaternionFromMatrixTest) { +- // Use a predetermined transformation matrix +- // to prevent running into division by zero. +- aiMatrix3x3 m, r; +- aiMatrix3x3::Translation(aiVector2D(14,-25), m); +- aiMatrix3x3::RotationZ(Math::aiPi() / 4.0f, r); +- m = m * r; +- +- result_cpp = aiQuaternion(m); +- aiCreateQuaternionFromMatrix(&result_c, &m); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiQuaternion, aiQuaternionFromEulerAnglesTest) { + const float x(RandPI.next()), + y(RandPI.next()), +@@ -115,14 +102,3 @@ TEST_F(AssimpAPITest_aiQuaternion, aiQuaternionMultiplyTest) { + aiQuaternionMultiply(&result_c, &temp); + EXPECT_EQ(result_cpp, result_c); + } +- +-TEST_F(AssimpAPITest_aiQuaternion, aiQuaternionInterpolateTest) { +- // Use predetermined quaternions to prevent division by zero +- // during slerp calculations. +- const float INTERPOLATION(0.5f); +- const auto q1 = aiQuaternion(aiVector3D(-1,1,1).Normalize(), Math::aiPi() / 4.0f); +- const auto q2 = aiQuaternion(aiVector3D(1,2,1).Normalize(), Math::aiPi() / 2.0f); +- aiQuaternion::Interpolate(result_cpp, q1, q2, INTERPOLATION); +- aiQuaternionInterpolate(&result_c, &q1, &q2, INTERPOLATION); +- EXPECT_EQ(result_cpp, result_c); +-} +--- a/test/unit/AssimpAPITest_aiVector2D.cpp ++++ b/test/unit/AssimpAPITest_aiVector2D.cpp +@@ -74,28 +74,6 @@ TEST_F(AssimpAPITest_aiVector2D, aiVector2AddTest) { + EXPECT_EQ(result_cpp, result_c); + } + +-TEST_F(AssimpAPITest_aiVector2D, aiVector2SubtractTest) { +- result_c = result_cpp = random_vec2(); +- result_cpp -= temp; +- aiVector2Subtract(&result_c, &temp); +- EXPECT_EQ(result_cpp, result_c); +-} +- +-TEST_F(AssimpAPITest_aiVector2D, aiVector2ScaleTest) { +- const float FACTOR = RandNonZero.next(); +- result_c = result_cpp = random_vec2(); +- result_cpp *= FACTOR; +- aiVector2Scale(&result_c, FACTOR); +- EXPECT_EQ(result_cpp, result_c); +-} +- +-TEST_F(AssimpAPITest_aiVector2D, aiVector2SymMulTest) { +- result_c = result_cpp = random_vec2(); +- result_cpp = result_cpp.SymMul(temp); +- aiVector2SymMul(&result_c, &temp); +- EXPECT_EQ(result_cpp, result_c); +-} +- + TEST_F(AssimpAPITest_aiVector2D, aiVector2DivideByScalarTest) { + const float DIVISOR = RandNonZero.next(); + result_c = result_cpp = random_vec2(); +-- +2.39.0 +