From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 316D51382C5 for ; Fri, 25 Dec 2020 15:05:45 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 62885E098F; Fri, 25 Dec 2020 15:05:44 +0000 (UTC) Received: from smtp.gentoo.org (dev.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 3ACA4E098F for ; Fri, 25 Dec 2020 15:05:44 +0000 (UTC) Received: from oystercatcher.gentoo.org (unknown [IPv6:2a01:4f8:202:4333:225:90ff:fed9:fc84]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id C7C1A341058 for ; Fri, 25 Dec 2020 15:05:41 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 4D2CBB8 for ; Fri, 25 Dec 2020 15:05:40 +0000 (UTC) From: "Andreas Sturmlechner" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Andreas Sturmlechner" Message-ID: <1608908707.bf8cb32d4ec64b747e310012e3af7fcb905fccf7.asturm@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: media-libs/libopenshot/, media-libs/libopenshot/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: media-libs/libopenshot/files/libopenshot-0.2.5-no-hwaccel-testfix.patch media-libs/libopenshot/libopenshot-0.2.5.ebuild X-VCS-Directories: media-libs/libopenshot/ media-libs/libopenshot/files/ X-VCS-Committer: asturm X-VCS-Committer-Name: Andreas Sturmlechner X-VCS-Revision: bf8cb32d4ec64b747e310012e3af7fcb905fccf7 X-VCS-Branch: master Date: Fri, 25 Dec 2020 15:05:40 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 1c1e9836-c126-4fca-adef-cf2b52009c89 X-Archives-Hash: 1230d33169e9d535660f3acee19a77df commit: bf8cb32d4ec64b747e310012e3af7fcb905fccf7 Author: Andreas Sturmlechner gentoo org> AuthorDate: Fri Dec 25 15:04:05 2020 +0000 Commit: Andreas Sturmlechner gentoo org> CommitDate: Fri Dec 25 15:05:07 2020 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bf8cb32d media-libs/libopenshot: Try to fix tests Minor: Fix Unquoted Variable Bug: https://bugs.gentoo.org/739638 Package-Manager: Portage-3.0.12, Repoman-3.0.2 Signed-off-by: Andreas Sturmlechner gentoo.org> .../libopenshot-0.2.5-no-hwaccel-testfix.patch | 45 ++++++++++++++++++++++ media-libs/libopenshot/libopenshot-0.2.5.ebuild | 5 ++- 2 files changed, 49 insertions(+), 1 deletion(-) diff --git a/media-libs/libopenshot/files/libopenshot-0.2.5-no-hwaccel-testfix.patch b/media-libs/libopenshot/files/libopenshot-0.2.5-no-hwaccel-testfix.patch new file mode 100644 index 00000000000..a079d9746c0 --- /dev/null +++ b/media-libs/libopenshot/files/libopenshot-0.2.5-no-hwaccel-testfix.patch @@ -0,0 +1,45 @@ +From f71051e8f1add0b893ffaa9a799625017978e7f8 Mon Sep 17 00:00:00 2001 +From: Frank Dana +Date: Thu, 20 Aug 2020 21:13:42 -0400 +Subject: [PATCH] Tests: Don't enable HW accel as side-effect (#557) + +The Settings test attempts to test the class by changing settings +variables and reading them back again. Problem is, that affects +the REST of the unit tests. So instead of enabling HW accel and +causing crashes, we'll diddle something innocuous, like OMP_THREADS. +--- + tests/Settings_Tests.cpp | 8 ++++---- + 1 file changed, 4 insertions(+), 4 deletions(-) + +diff --git a/tests/Settings_Tests.cpp b/tests/Settings_Tests.cpp +index b63b56a8..65bd66a6 100644 +--- a/tests/Settings_Tests.cpp ++++ b/tests/Settings_Tests.cpp +@@ -41,7 +41,7 @@ TEST(Settings_Default_Constructor) + // Create an empty color + Settings *s = Settings::Instance(); + +- CHECK_EQUAL(0, s->HARDWARE_DECODER); ++ CHECK_EQUAL(12, s->OMP_THREADS); + CHECK_EQUAL(false, s->HIGH_QUALITY_SCALING); + CHECK_EQUAL(false, s->WAIT_FOR_VIDEO_PROCESSING_TASK); + } +@@ -50,15 +50,15 @@ TEST(Settings_Change_Settings) + { + // Create an empty color + Settings *s = Settings::Instance(); +- s->HARDWARE_DECODER = 1; ++ s->OMP_THREADS = 8; + s->HIGH_QUALITY_SCALING = true; + s->WAIT_FOR_VIDEO_PROCESSING_TASK = true; + +- CHECK_EQUAL(1, s->HARDWARE_DECODER); ++ CHECK_EQUAL(8, s->OMP_THREADS); + CHECK_EQUAL(true, s->HIGH_QUALITY_SCALING); + CHECK_EQUAL(true, s->WAIT_FOR_VIDEO_PROCESSING_TASK); + +- CHECK_EQUAL(1, s->HARDWARE_DECODER); ++ CHECK_EQUAL(8, Settings::Instance()->OMP_THREADS); + CHECK_EQUAL(true, Settings::Instance()->HIGH_QUALITY_SCALING); + CHECK_EQUAL(true, Settings::Instance()->WAIT_FOR_VIDEO_PROCESSING_TASK); + } diff --git a/media-libs/libopenshot/libopenshot-0.2.5.ebuild b/media-libs/libopenshot/libopenshot-0.2.5.ebuild index fdfa393fcfc..37973900308 100644 --- a/media-libs/libopenshot/libopenshot-0.2.5.ebuild +++ b/media-libs/libopenshot/libopenshot-0.2.5.ebuild @@ -38,7 +38,10 @@ BDEPEND=" test? ( dev-libs/unittest++ ) " -PATCHES=( ${FILESDIR}/${PN}-0.2.5-gcc10.patch ) +PATCHES=( + "${FILESDIR}/${P}-gcc10.patch" + "${FILESDIR}/${P}-no-hwaccel-testfix.patch" +) check_compiler() { if [[ ${MERGE_TYPE} != binary ]] && ! tc-has-openmp; then