From: "Conrad Kostecki" <conikost@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: kde-apps/gwenview/files/
Date: Sun, 23 Mar 2025 14:30:27 +0000 (UTC) [thread overview]
Message-ID: <1742740203.c3606034c797c49e4b4d844e44cea8a655449f68.conikost@gentoo> (raw)
commit: c3606034c797c49e4b4d844e44cea8a655449f68
Author: Michael Mair-Keimberger <mmk <AT> levelnine <DOT> at>
AuthorDate: Wed Mar 12 07:22:37 2025 +0000
Commit: Conrad Kostecki <conikost <AT> gentoo <DOT> org>
CommitDate: Sun Mar 23 14:30:03 2025 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c3606034
kde-apps/gwenview: remove unused patch(es)
Signed-off-by: Michael Mair-Keimberger <mmk <AT> levelnine.at>
Signed-off-by: Conrad Kostecki <conikost <AT> gentoo.org>
kde-apps/gwenview/files/gwenview-24.08.0-odr.patch | 58 ----------------------
1 file changed, 58 deletions(-)
diff --git a/kde-apps/gwenview/files/gwenview-24.08.0-odr.patch b/kde-apps/gwenview/files/gwenview-24.08.0-odr.patch
deleted file mode 100644
index 148921f18169..000000000000
--- a/kde-apps/gwenview/files/gwenview-24.08.0-odr.patch
+++ /dev/null
@@ -1,58 +0,0 @@
-From b5cc214574eeb62f469a058daf52cd92270e4aa9 Mon Sep 17 00:00:00 2001
-From: Eli Schwartz <eschwartz@gentoo.org>
-Date: Tue, 3 Sep 2024 00:40:11 -0400
-Subject: [PATCH] fix ODR violation: many files had the wrong definition of
- Enum
-MIME-Version: 1.0
-Content-Type: text/plain; charset=UTF-8
-Content-Transfer-Encoding: 8bit
-
-This enum has a different number of values, depending on whether
-gwenview is configured with the semanticinfo backend "none" or
-fake/baloo.
-
-In order to avoid Undefined Behavior, every file that includes sorting.h
-also has to include config-gwenview.h *before* sorting.h, which was not
-true. Solve this by including the config header inside the sorting.h
-header.
-
-Fixes link failures when LTO is enabled, since LTO allows the compiler
-to do whole-program optimization and type-check across the entire
-program:
-
-```
-/var/tmp/portage/kde-apps/gwenview-24.08.0/work/gwenview-24.08.0/app/../lib/sorting.h:37:6: error: type ‘Gwenview::Sorting::Enum’ violates the C++ One Definition Rule [-Werror=odr]
- 37 | enum Enum {
- | ^
-/var/tmp/portage/kde-apps/gwenview-24.08.0/work/gwenview-24.08.0/app/../lib/sorting.h:37:6: note: an enum with different number of values is defined in another translation unit
- 37 | enum Enum {
- | ^
-lto1: some warnings being treated as errors
-lto-wrapper: fatal error: /usr/bin/x86_64-pc-linux-gnu-g++ returned 1 exit status
-compilation terminated.
-/usr/libexec/gcc/x86_64-pc-linux-gnu/ld: error: lto-wrapper failed
-collect2: error: ld returned 1 exit status
-```
-
-Fixes: https://bugs.gentoo.org/921649
----
- lib/sorting.h | 3 +++
- 1 file changed, 3 insertions(+)
-
-diff --git a/lib/sorting.h b/lib/sorting.h
-index 529ea6fc9..c008865cf 100644
---- a/lib/sorting.h
-+++ b/lib/sorting.h
-@@ -21,6 +21,9 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Cambridge, MA 02110-1301, USA
- #ifndef SORTING_H
- #define SORTING_H
-
-+// Do not assume every file that includes this one, includes the config.h
-+#include <config-gwenview.h>
-+
- // Qt
-
- // KF
---
-GitLab
-
next reply other threads:[~2025-03-23 14:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-23 14:30 Conrad Kostecki [this message]
-- strict thread matches above, loose matches on Subject: below --
2019-06-16 21:57 [gentoo-commits] repo/gentoo:master commit in: kde-apps/gwenview/files/ Andreas Sturmlechner
2019-01-17 0:36 Andreas Sturmlechner
2018-09-08 12:51 Andreas Sturmlechner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1742740203.c3606034c797c49e4b4d844e44cea8a655449f68.conikost@gentoo \
--to=conikost@gentoo.org \
--cc=gentoo-commits@lists.gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox