Skip to content

Commit

Permalink
kde-frameworks/kio: Fix build with USE -X, drop obsolete DEPENDs
Browse files Browse the repository at this point in the history
Upstream commit c7a277e6810b74b7f664b029573beab592932421

Bug: https://bugs.gentoo.org/813450
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
  • Loading branch information
a17r authored and Your Name committed Aug 9, 2022
1 parent ca92ea2 commit 1438d4a
Show file tree
Hide file tree
Showing 3 changed files with 78 additions and 14 deletions.
72 changes: 72 additions & 0 deletions kde-frameworks/kio/files/kio-5.96.0-with_x11.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
From c7a277e6810b74b7f664b029573beab592932421 Mon Sep 17 00:00:00 2001
From: Andreas Sturmlechner <asturm@gentoo.org>
Date: Mon, 1 Aug 2022 22:10:02 +0200
Subject: [PATCH] Drop obsolete X11 dependency, introduce WITH_X11 option
instead

This will only determine whether to search for Qt5X11Extras and then
set HAVE_X11 accordingly. This is a behavior change as previously it was
silently disabled if X11 was not found.

We want to be able to build without X11 support even if some of the used
libraries may not work w/o X11 themselves yet or need to be built with
X11 support for other reverse dependencies.

HAVE_X11 already exists and is set automagically so far, but using
-DCMAKE_DISABLE_FIND_PACKAGE_X11 will break if any dependencies list X11
as required in their cmake config.

Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
---
CMakeLists.txt | 19 +++++++++----------
src/ioslaves/http/CMakeLists.txt | 3 ---
2 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 95b5db299..c3372f7ea 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -117,17 +117,16 @@ set_package_properties(GSSAPI PROPERTIES DESCRIPTION "Allows KIO to make use of
PURPOSE "A MIT or HEIMDAL flavor of GSSAPI can be used"
)

-if (NOT APPLE AND NOT WIN32)
- find_package(X11)
-endif()
-
-set(HAVE_X11 ${X11_FOUND})
-if (HAVE_X11)
- if (QT_MAJOR_VERSION STREQUAL "5")
- find_package(Qt5 ${REQUIRED_QT_VERSION} CONFIG REQUIRED X11Extras)
- else()
- # qtx11extras_p.h is in Qt6Gui, which is implied by the find_package(QtWidgets) above
+if (NOT APPLE AND NOT WIN32 AND NOT ANDROID)
+ option(WITH_X11 "Build with X11 integration" ON)
+ if(WITH_X11)
+ if (QT_MAJOR_VERSION STREQUAL "5")
+ find_package(Qt5 ${REQUIRED_QT_VERSION} CONFIG REQUIRED X11Extras)
+ else()
+ # qtx11extras_p.h is in Qt6Gui, which is implied by the find_package(QtWidgets) above
+ endif()
endif()
+ set(HAVE_X11 ${WITH_X11})
endif()

find_package(ACL)
diff --git a/src/ioslaves/http/CMakeLists.txt b/src/ioslaves/http/CMakeLists.txt
index 4fbddb4a3..689f63dd8 100644
--- a/src/ioslaves/http/CMakeLists.txt
+++ b/src/ioslaves/http/CMakeLists.txt
@@ -3,9 +3,6 @@ include(ECMMarkNonGuiExecutable)
include(ConfigureChecks.cmake)
configure_file(config-kioslave-http.h.cmake ${CMAKE_CURRENT_BINARY_DIR}/config-kioslave-http.h )

-find_package(X11)
-set(HAVE_X11 ${X11_FOUND})
-
if(GSSAPI_FOUND)
set(HAVE_LIBGSSAPI 1)
if(GSSAPI_FLAVOR STREQUAL "MIT")
--
GitLab

11 changes: 4 additions & 7 deletions kde-frameworks/kio/kio-5.97.0.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ RDEPEND="
=kde-frameworks/kservice-${PVCUT}*:5
=kde-frameworks/ktextwidgets-${PVCUT}*:5
=kde-frameworks/kwidgetsaddons-${PVCUT}*:5
=kde-frameworks/kwindowsystem-${PVCUT}*:5
=kde-frameworks/kwindowsystem-${PVCUT}*:5[X?]
=kde-frameworks/kxmlgui-${PVCUT}*:5
=kde-frameworks/solid-${PVCUT}*:5
acl? (
Expand All @@ -62,22 +62,19 @@ RDEPEND="
DEPEND="${RDEPEND}
>=dev-qt/qtconcurrent-${QTMIN}:5
test? ( sys-libs/zlib )
X? (
x11-base/xorg-proto
x11-libs/libX11
x11-libs/libXrender
)
"
PDEPEND=">=kde-frameworks/kded-${PVCUT}:5"

PATCHES=( "${FILESDIR}/${PN}-5.96.0-with_x11.patch" )

src_configure() {
local mycmakeargs=(
-DKIO_NO_PUBLIC_QTCONCURRENT=ON
$(cmake_use_find_package acl ACL)
$(cmake_use_find_package handbook KF5DocTools)
$(cmake_use_find_package kerberos GSSAPI)
$(cmake_use_find_package kwallet KF5Wallet)
$(cmake_use_find_package X X11)
-DWITH_X11=$(usex X)
)

ecm_src_configure
Expand Down
9 changes: 2 additions & 7 deletions kde-frameworks/kio/kio-9999.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ RDEPEND="
=kde-frameworks/kservice-${PVCUT}*:5
=kde-frameworks/ktextwidgets-${PVCUT}*:5
=kde-frameworks/kwidgetsaddons-${PVCUT}*:5
=kde-frameworks/kwindowsystem-${PVCUT}*:5
=kde-frameworks/kwindowsystem-${PVCUT}*:5[X?]
=kde-frameworks/kxmlgui-${PVCUT}*:5
=kde-frameworks/solid-${PVCUT}*:5
acl? (
Expand All @@ -62,11 +62,6 @@ RDEPEND="
DEPEND="${RDEPEND}
>=dev-qt/qtconcurrent-${QTMIN}:5
test? ( sys-libs/zlib )
X? (
x11-base/xorg-proto
x11-libs/libX11
x11-libs/libXrender
)
"
PDEPEND=">=kde-frameworks/kded-${PVCUT}:5"

Expand All @@ -77,7 +72,7 @@ src_configure() {
$(cmake_use_find_package handbook KF5DocTools)
$(cmake_use_find_package kerberos GSSAPI)
$(cmake_use_find_package kwallet KF5Wallet)
$(cmake_use_find_package X X11)
-DWITH_X11=$(usex X)
)

ecm_src_configure
Expand Down

0 comments on commit 1438d4a

Please sign in to comment.