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.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id B205015808B for ; Thu, 10 Feb 2022 09:13:27 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 7EE05E0903; Thu, 10 Feb 2022 09:13:26 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 3F899E0903 for ; Thu, 10 Feb 2022 09:13:26 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id E57783430EF for ; Thu, 10 Feb 2022 09:13:24 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 384C52D6 for ; Thu, 10 Feb 2022 09:13:22 +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: <1644484257.448ebc67b241e315aaf6b7f1f422a6993a204b94.asturm@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: kde-frameworks/kiconthemes/files/, kde-frameworks/kiconthemes/ X-VCS-Repository: repo/gentoo X-VCS-Files: kde-frameworks/kiconthemes/Manifest kde-frameworks/kiconthemes/files/kiconthemes-5.88.0-fix-icon-preference.patch kde-frameworks/kiconthemes/kiconthemes-5.88.0-r1.ebuild X-VCS-Directories: kde-frameworks/kiconthemes/files/ kde-frameworks/kiconthemes/ X-VCS-Committer: asturm X-VCS-Committer-Name: Andreas Sturmlechner X-VCS-Revision: 448ebc67b241e315aaf6b7f1f422a6993a204b94 X-VCS-Branch: master Date: Thu, 10 Feb 2022 09:13:22 +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: a4442984-7945-4c90-9787-a92acec3cbf8 X-Archives-Hash: 8dfe835349e2b2da742d217334c3d3d0 commit: 448ebc67b241e315aaf6b7f1f422a6993a204b94 Author: Andreas Sturmlechner gentoo org> AuthorDate: Thu Feb 10 08:20:43 2022 +0000 Commit: Andreas Sturmlechner gentoo org> CommitDate: Thu Feb 10 09:10:57 2022 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=448ebc67 kde-frameworks/kiconthemes: drop 5.88.0* Signed-off-by: Andreas Sturmlechner gentoo.org> kde-frameworks/kiconthemes/Manifest | 1 - .../kiconthemes-5.88.0-fix-icon-preference.patch | 122 --------------------- .../kiconthemes/kiconthemes-5.88.0-r1.ebuild | 34 ------ 3 files changed, 157 deletions(-) diff --git a/kde-frameworks/kiconthemes/Manifest b/kde-frameworks/kiconthemes/Manifest index bd16752e21fd..bc176393a03b 100644 --- a/kde-frameworks/kiconthemes/Manifest +++ b/kde-frameworks/kiconthemes/Manifest @@ -1,2 +1 @@ -DIST kiconthemes-5.88.0.tar.xz 223284 BLAKE2B 374935539c45468dd60364701ad2ca9ba1b7bc5852692a4299d74efd99fe615913626e42525deeebc5ea27ae44aab7795ffbaefa2978f3a214c3a3359e1cc664 SHA512 56bb3da876cfe7bc4999730ae91c9e71001aff63c904c4bdeb5989b6465c981b848af36942a5a5979e08af0b654c9e403210b037cab7b085d14281ed769650c9 DIST kiconthemes-5.90.0.tar.xz 225884 BLAKE2B 9697b1cda2520807b6b21e9210a6e436cfa9cbba25ce3a05c339c94dacf1687700d8ed59c806e222e811df166856217b5080d311a44eedf375b252e048bb7d64 SHA512 91a281df7c0336b1358969d0c6ef2899679e1b0088d74f5ad5c02ddcc9d0e0bfc71ea6fe13eec6ba6668996128ef6d175b94ae46388327534a28f851eae99998 diff --git a/kde-frameworks/kiconthemes/files/kiconthemes-5.88.0-fix-icon-preference.patch b/kde-frameworks/kiconthemes/files/kiconthemes-5.88.0-fix-icon-preference.patch deleted file mode 100644 index e8b48b468323..000000000000 --- a/kde-frameworks/kiconthemes/files/kiconthemes-5.88.0-fix-icon-preference.patch +++ /dev/null @@ -1,122 +0,0 @@ -From 13181b03eac3c85f0649d5399d8c3037c388928c Mon Sep 17 00:00:00 2001 -From: Jan Blackquill -Date: Thu, 25 Nov 2021 14:51:06 -0500 -Subject: [PATCH] KIconLoader: prefer icons from current theme before falling - back to other themes - -BUG: 445804 ---- - autotests/kiconloader_unittest.cpp | 25 ++++++++++++++++++++++ - src/kiconloader.cpp | 34 ++++++------------------------ - 2 files changed, 32 insertions(+), 27 deletions(-) - -diff --git a/autotests/kiconloader_unittest.cpp b/autotests/kiconloader_unittest.cpp -index 813215d..c232111 100644 ---- a/autotests/kiconloader_unittest.cpp -+++ b/autotests/kiconloader_unittest.cpp -@@ -112,6 +112,12 @@ private Q_SLOTS: - QVERIFY(QFile::copy(QStringLiteral(":/test-22x22.png"), testIconsDir.filePath(QStringLiteral("breeze/22x22/mimetypes/unknown.png")))); - QVERIFY(QFile::copy(QStringLiteral(":/coloredsvgicon.svg"), testIconsDir.filePath(QStringLiteral("breeze/22x22/apps/coloredsvgicon.svg")))); - -+ // prepare some icons for our actions test -+ // when querying breeze for 'one-two', we expect -+ // 'one' from breeze instead of oxygen's 'one-two'. -+ QVERIFY(QFile::copy(QStringLiteral(":/test-22x22.png"), testIconsDir.filePath(QStringLiteral("oxygen/22x22/actions/one-two.png")))); -+ QVERIFY(QFile::copy(QStringLiteral(":/test-22x22.png"), testIconsDir.filePath(QStringLiteral("breeze/22x22/actions/one.png")))); -+ - QVERIFY(QFile::setPermissions(breezeThemeFile, QFileDevice::ReadOwner | QFileDevice::WriteOwner)); - KConfig configFile(breezeThemeFile); - KConfigGroup iconThemeGroup = configFile.group("Icon Theme"); -@@ -332,6 +338,25 @@ private Q_SLOTS: - QVERIFY(QFile::exists(unknownPath)); - } - -+ void testCorrectFallback() -+ { -+ // we want to prefer icons from the same theme -+ -+ // so if we have something like: -+ /* -+ oxygen: -+ one-two -+ -+ breeze: -+ one -+ */ -+ // and we ask for 'one-two', we expect to see 'one' from breeze instead -+ // of 'one-two' from oxygen. -+ QString path; -+ KIconLoader::global()->loadIcon(QStringLiteral("one-two"), KIconLoader::Desktop, 24, KIconLoader::DefaultState, QStringList(), &path); -+ QVERIFY(path.contains("breeze/22x22/actions")); -+ } -+ - void testPathStore() - { - QString path; -diff --git a/src/kiconloader.cpp b/src/kiconloader.cpp -index 4d4181d..8a644d4 100644 ---- a/src/kiconloader.cpp -+++ b/src/kiconloader.cpp -@@ -1021,12 +1021,7 @@ QString KIconLoaderPrivate::findMatchingIcon(const QString &name, int size, qrea - { - const_cast(this)->initIconThemes(); - -- // Do two passes through themeNodes. -- // -- // The first pass looks for an exact match in each themeNode one after the other. -- // If one is found and it is an app icon then return that icon. -- // -- // In the next pass (assuming the first pass failed), it looks for -+ // This looks for the exact match and its - // generic fallbacks in each themeNode one after the other. - - // In theory we should only do this for mimetype icons, not for app icons, -@@ -1036,22 +1031,17 @@ QString KIconLoaderPrivate::findMatchingIcon(const QString &name, int size, qrea - // Once everyone uses that to look up mimetype icons, we can kill the fallback code - // from this method. - -- for (KIconThemeNode *themeNode : std::as_const(links)) { -- const QString path = themeNode->theme->iconPathByName(name, size, KIconLoader::MatchBest, scale); -- if (!path.isEmpty()) { -- return path; -- } -- } -- -- if (name.endsWith(QLatin1String("-x-generic"))) { -- return QString(); // no further fallback -- } -- bool genericFallback = false; -+ bool genericFallback = name.endsWith(QLatin1String("-x-generic"));; - QString path; - for (KIconThemeNode *themeNode : std::as_const(links)) { - QString currentName = name; - - while (!currentName.isEmpty()) { -+ path = themeNode->theme->iconPathByName(currentName, size, KIconLoader::MatchBest, scale); -+ if (!path.isEmpty()) { -+ return path; -+ } -+ - if (genericFallback) { - // we already tested the base name - break; -@@ -1088,16 +1078,6 @@ QString KIconLoaderPrivate::findMatchingIcon(const QString &name, int size, qrea - break; - } - } -- -- if (currentName.isEmpty()) { -- break; -- } -- -- // qCDebug(KICONTHEMES) << "Looking up" << currentName; -- path = themeNode->theme->iconPathByName(currentName, size, KIconLoader::MatchBest, scale); -- if (!path.isEmpty()) { -- return path; -- } - } - } - --- -GitLab - diff --git a/kde-frameworks/kiconthemes/kiconthemes-5.88.0-r1.ebuild b/kde-frameworks/kiconthemes/kiconthemes-5.88.0-r1.ebuild deleted file mode 100644 index 810f917fbbf5..000000000000 --- a/kde-frameworks/kiconthemes/kiconthemes-5.88.0-r1.ebuild +++ /dev/null @@ -1,34 +0,0 @@ -# Copyright 1999-2021 Gentoo Authors -# Distributed under the terms of the GNU General Public License v2 - -EAPI=8 - -ECM_DESIGNERPLUGIN="true" -PVCUT=$(ver_cut 1-2) -QTMIN=5.15.2 -VIRTUALX_REQUIRED="test" -inherit ecm kde.org - -DESCRIPTION="Framework for icon theming and configuration" -LICENSE="LGPL-2+" -KEYWORDS="amd64 ~arm arm64 ~ppc64 ~riscv x86" -IUSE="" - -RESTRICT="test" # bug 574770 - -DEPEND=" - >=dev-qt/qtdbus-${QTMIN}:5 - >=dev-qt/qtgui-${QTMIN}:5 - >=dev-qt/qtsvg-${QTMIN}:5 - >=dev-qt/qtwidgets-${QTMIN}:5 - =kde-frameworks/karchive-${PVCUT}*:5 - =kde-frameworks/kconfig-${PVCUT}*:5 - =kde-frameworks/kconfigwidgets-${PVCUT}*:5 - =kde-frameworks/kcoreaddons-${PVCUT}*:5 - =kde-frameworks/ki18n-${PVCUT}*:5 - =kde-frameworks/kitemviews-${PVCUT}*:5 - =kde-frameworks/kwidgetsaddons-${PVCUT}*:5 -" -RDEPEND="${DEPEND}" - -PATCHES=( "${FILESDIR}"/${P}-fix-icon-preference.patch ) # KDE-bug 445804