Gentoo Archives: gentoo-commits

From: Andreas Sturmlechner <asturm@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] proj/kde:master commit in: kde-frameworks/kdelibs4support/files/, kde-frameworks/kdelibs4support/
Date: Wed, 02 Nov 2022 12:59:07
Message-Id: 1667393935.39d720bca9a1a9e89859dbbf3f91c7d10fe6d755.asturm@gentoo
1 commit: 39d720bca9a1a9e89859dbbf3f91c7d10fe6d755
2 Author: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
3 AuthorDate: Wed Nov 2 10:25:10 2022 +0000
4 Commit: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
5 CommitDate: Wed Nov 2 12:58:55 2022 +0000
6 URL: https://gitweb.gentoo.org/proj/kde.git/commit/?id=39d720bc
7
8 kde-frameworks/kdelibs4support: Drop upstreamed patch
9
10 Upstream commit 598dcb3b1549230c931bb6dcf9001b395f8ec9f4
11
12 Signed-off-by: Andreas Sturmlechner <asturm <AT> gentoo.org>
13
14 .../files/kdelibs4support-5.99.0-with_x11.patch | 98 ----------------------
15 .../kdelibs4support/kdelibs4support-9999.ebuild | 3 -
16 2 files changed, 101 deletions(-)
17
18 diff --git a/kde-frameworks/kdelibs4support/files/kdelibs4support-5.99.0-with_x11.patch b/kde-frameworks/kdelibs4support/files/kdelibs4support-5.99.0-with_x11.patch
19 deleted file mode 100644
20 index 24bcb1b9cc..0000000000
21 --- a/kde-frameworks/kdelibs4support/files/kdelibs4support-5.99.0-with_x11.patch
22 +++ /dev/null
23 @@ -1,98 +0,0 @@
24 -From 026571f37c990007be3d7e536b335c19ab98009e Mon Sep 17 00:00:00 2001
25 -From: Andreas Sturmlechner <asturm@g.o>
26 -Date: Sat, 6 Aug 2022 11:16:00 +0200
27 -Subject: [PATCH] Add CMake option to build WITH_X11
28 -
29 -We want to be able to build without X11 support even if some of the used
30 -libraries may not work w/o X11 themselves yet or need to be built with
31 -X11 support for other reverse dependencies.
32 -
33 -HAVE_X11 already exists and is set automagically so far, but using
34 --DCMAKE_DISABLE_FIND_PACKAGE_X11 will break if any dependencies list X11
35 -as required in their cmake config. This is a behavior change as previously
36 -it was silently disabled if X11 was not found.
37 -
38 -Signed-off-by: Andreas Sturmlechner <asturm@g.o>
39 ----
40 - CMakeLists.txt | 18 +++++++++++-------
41 - src/CMakeLists.txt | 6 +++---
42 - tests/CMakeLists.txt | 2 +-
43 - 3 files changed, 15 insertions(+), 11 deletions(-)
44 -
45 -diff --git a/CMakeLists.txt b/CMakeLists.txt
46 -index f577df45..378d23d4 100644
47 ---- a/CMakeLists.txt
48 -+++ b/CMakeLists.txt
49 -@@ -51,14 +51,18 @@ if(WIN32)
50 - endif()
51 -
52 - if (NOT APPLE AND NOT WIN32)
53 -- find_package(X11)
54 -+ option(WITH_X11 "Build with X11 integration and session management" ON)
55 -+ if(WITH_X11)
56 -+ find_package(X11)
57 -+ set(HAVE_X11 ${X11_FOUND})
58 -+ if (HAVE_X11)
59 -+ add_feature_info("X11 Session Management (libSM)" X11_SM_FOUND "Support for session management in KApplication")
60 -+ find_package(Qt5 ${REQUIRED_QT_VERSION} CONFIG REQUIRED X11Extras)
61 -+ find_package(XCB REQUIRED COMPONENTS XCB)
62 -+ endif()
63 -+ endif()
64 - endif()
65 --set(HAVE_X11 ${X11_FOUND})
66 --if (HAVE_X11)
67 -- add_feature_info("X11 Session Management (libSM)" X11_SM_FOUND "Support for session management in KApplication")
68 -- find_package(Qt5 ${REQUIRED_QT_VERSION} CONFIG REQUIRED X11Extras)
69 -- find_package(XCB REQUIRED COMPONENTS XCB)
70 --endif ()
71 -+
72 - if (APPLE)
73 - cmake_find_frameworks(CoreFoundation)
74 - endif ()
75 -diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
76 -index 2096b355..4649a9d5 100644
77 ---- a/src/CMakeLists.txt
78 -+++ b/src/CMakeLists.txt
79 -@@ -226,7 +226,7 @@ if (NOT KIO_NO_NEPOMUK)
80 - )
81 - endif ()
82 -
83 --if (X11_FOUND)
84 -+if (HAVE_X11)
85 - set(libkdelibs4support_SRCS ${libkdelibs4support_SRCS} kdeui/kxerrorhandler.cpp )
86 - list(APPEND libkdelibs4support_OPTIONAL_LIBS Qt5::X11Extras ${X11_LIBRARIES})
87 - endif ()
88 -@@ -312,7 +312,7 @@ set(platformLinkLibraries)
89 - if (APPLE)
90 - set(platformLinkLibraries "-framework CoreFoundation -framework Carbon -lresolv")
91 - endif()
92 --if (X11_FOUND)
93 -+if (HAVE_X11)
94 - set(platformLinkLibraries ${XCB_XCB_LIBRARY})
95 - endif()
96 -
97 -@@ -545,7 +545,7 @@ install( FILES ${CMAKE_CURRENT_BINARY_DIR}/all_languages DESTINATION ${KDE_INSTA
98 - install( FILES kdeui/40.colors kdeui/Oxygen.colors kdeui/Web.colors kdeui/Royal.colors kdeui/Rainbow.colors
99 - DESTINATION ${KDE_INSTALL_CONFDIR}/colors)
100 -
101 --if (NOT X11_FOUND)
102 -+if (NOT HAVE_X11)
103 - # install our copy of the rgb.txt named colors list on systems without X11
104 - install(FILES kdeui/rgb.txt DESTINATION ${KDE_INSTALL_DATADIR_KF5}/kdeui)
105 - endif ()
106 -diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
107 -index 53f870ff..db4b28db 100644
108 ---- a/tests/CMakeLists.txt
109 -+++ b/tests/CMakeLists.txt
110 -@@ -63,7 +63,7 @@ target_link_libraries(networkingservice KF5::KDELibs4Support)
111 -
112 - ###### kxerrorhandlertest #####
113 -
114 --if (X11_FOUND)
115 -+if (HAVE_X11)
116 - find_package(Qt5X11Extras ${REQUIRED_QT_VERSION} REQUIRED NO_MODULE)
117 - kdelibs4support_examples( kxerrorhandlertest )
118 -
119 ---
120 -2.35.1
121 -
122
123 diff --git a/kde-frameworks/kdelibs4support/kdelibs4support-9999.ebuild b/kde-frameworks/kdelibs4support/kdelibs4support-9999.ebuild
124 index b455381245..fd04667df1 100644
125 --- a/kde-frameworks/kdelibs4support/kdelibs4support-9999.ebuild
126 +++ b/kde-frameworks/kdelibs4support/kdelibs4support-9999.ebuild
127 @@ -83,9 +83,6 @@ PATCHES=(
128 # downstream patches
129 "${FILESDIR}/${PN}-5.80.0-no-kdesignerplugin.patch" # bug 755956
130 "${FILESDIR}/${PN}-5.86.0-unused-dep.patch" # bug 755956
131 - # pending upstream:
132 - # https://invent.kde.org/frameworks/kdelibs4support/-/merge_requests/21
133 - "${FILESDIR}/${PN}-5.99.0-with_x11.patch"
134 )
135
136 src_prepare() {