Bug 930943

Summary:	net-misc/nextcloud-client-3.13.0 fails to configure if both 5 and 6 slots of kde-frameworks/kio are installed
Product:	Gentoo Linux	Reporter:	Andrei Slavoiu <ansla80>
Component:	Current packages	Assignee:	Bernard Cafarelli <voyageur>
Status:	RESOLVED FIXED
Severity:	normal	CC:	adel.ks, cctsurf, cyber+gentoo, flow, freddie, gentoo_bugs.nu_q5v, gerion.entrup, kripton, me, michael, seb, voyageur
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log emerge --info build.log emerge --info 0001-Fix-build-with-KF6.patch

Description Andrei Slavoiu 2024-04-29 19:41:51 UTC

It looks like first stage of Qt 6 support was added in this version, should probably disable Qt 6 support for the shell integration until full Qt 6 support lands.

For now it fails like this when both versions are available:
CMake Error at /usr/lib64/cmake/Qt6Qml/Qt6QmlMacros.cmake:2829 (message):
  qt_generate_foreign_qml_types() is only available in Qt 6.
Call Stack (most recent call first):
  /usr/lib64/cmake/Qt6Qml/Qt6QmlConfig.cmake:145 (include)
  /usr/share/cmake/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
  /usr/lib64/cmake/KF6Config/KF6ConfigConfig.cmake:53 (find_dependency)
  /usr/share/cmake/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
  /usr/lib64/cmake/KF6KIO/KF6KIOConfig.cmake:41 (find_dependency)
  shell_integration/CMakeLists.txt:15 (find_package)


Reproducible: Always

Comment 1 Andrei Slavoiu 2024-04-29 19:42:47 UTC

Created attachment 891973 [details]
build.log

Comment 2 Andrei Slavoiu 2024-04-29 19:44:34 UTC

Created attachment 891974 [details]
emerge --info

Comment 3 Adel KARA SLIMANE 2024-05-05 02:12:04 UTC

Created attachment 892275 [details]
build.log

Comment 4 Adel KARA SLIMANE 2024-05-05 02:13:04 UTC

Created attachment 892276 [details]
emerge --info

Comment 5 Adel KARA SLIMANE 2024-05-05 02:14:07 UTC

Hello,

Hitting against the same issue. Thanks!

[ebuild     U ] net-misc/nextcloud-client-3.13.0 [3.12.3] USE="dolphin -doc -nautilus -test -webengine"

Comment 6 Sébastien Monassa 2024-05-07 09:42:34 UTC

Confirmed here with unmasked Plasma 6.

Comment 7 gerion 2024-05-07 13:56:12 UTC

Applying this PR fixes the issue:
https://github.com/nextcloud/desktop/pull/6599/

See the attached patch. Adding it to /etc/portage/patches/net-misc/nextcloud-client works.

Comment 8 gerion 2024-05-07 14:02:36 UTC

Created attachment 892442 [details, diff]
0001-Fix-build-with-KF6.patch

Comment 9 Bernard Cafarelli gentoo-dev

2024-05-07 14:14:10 UTC

Thanks for the detailed report all, the switch to Qt6 will be interesting to sort out!

In the meantime, let me check that patch - I guess the change to   qt5_add_resources means it will need some rework for qt6, but again this is for later

Comment 10 Anna Vyalkova 2024-07-09 08:58:21 UTC

Configure fails only when "dolphin" USE-flag is enabled

Comment 11 James 2024-07-12 21:04:39 UTC

(In reply to gerion from comment #8)
> Created attachment 892442 [details, diff] [details, diff]
> 0001-Fix-build-with-KF6.patch

This patch also works on v. 3.13.2

Comment 12 Bernard Cafarelli gentoo-dev

2024-07-16 20:23:33 UTC

I am watching upstream bug so let's see there about need for qt6_add_resources

In the meantime, adding the current patch to 3.13.2, sorry for the delay

Comment 13 Larry the Git Cow gentoo-dev

2024-07-16 20:27:13 UTC

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b0c1e8c85956eeb654ebdaddfd399ab0f24cf17b

commit b0c1e8c85956eeb654ebdaddfd399ab0f24cf17b
Author:     Bernard Cafarelli <voyageur@gentoo.org>
AuthorDate: 2024-07-16 20:25:43 +0000
Commit:     Bernard Cafarelli <voyageur@gentoo.org>
CommitDate: 2024-07-16 20:27:10 +0000

    net-misc/nextcloud-client: fix build if kio:6 is installed
    
    Closes: https://bugs.gentoo.org/930943
    Signed-off-by: Bernard Cafarelli <voyageur@gentoo.org>

 .../nextcloud-client-3.13.2-fix_build_kf6.patch    | 38 ++++++++++++++++++++++
 .../nextcloud-client-3.13.2.ebuild                 |  1 +
 2 files changed, 39 insertions(+)