Bug 462660

Summary:	kwin commit a6a022f00 breaks build: src/group.cpp:11:10 KX11Extras: No such file or directory
Product:	[Plasma] kwin	Reporter:	Duncan <1i5t5.duncan>
Component:	core	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nicolas.fella, vlad.zahorodnii
Priority:	NOR
Version First Reported In:	git master
Target Milestone:	---
Platform:	Other
OS:	Other
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Duncan 2022-12-05 07:08:05 UTC

On git master using gentoo/kde's live-git ebuilds. 

commit a6a022f00
Author:     Nicolas Fella [snipped email]
AuthorDate: Thu Dec 1 02:15:26 2022 +0100
Commit:     Vlad Zahorodnii [snipped email
CommitDate: Thu Dec 1 08:08:54 2022 +0000

Port away from deprecated KWindowSystem API

... breaks the build with #Include <KX11Extras> on line 18.

Additional information:  I no longer have an independent X/xorg installed, only xwayland, and am gradually reducing the number of installed X-related dependencies as they become optional (USE=-X in gentoo-speak).  Presumably KX11Extras is a (previously) optional X-related component that I have turned off.  Preferably it'd be optional here too, but regardless, there's apparently no cmake find for it here or I'd be getting that error instead of the build error:

FAILED: src/CMakeFiles/kwin.dir/group.cpp.o
...
src/group.cpp:18:10: fatal error: KX11Extras: No such file or directory
18 | #include <KX11Extras>
 |          ^~~~~~~~~~~~
compilation terminated.

Comment 1 Duncan 2022-12-05 07:11:06 UTC

CCing commit author and committer

Comment 2 Duncan 2022-12-05 07:34:35 UTC

Double-confirmed, both by building the previous commit and not being able to build that one, and by applying the reversing patch at current head 1a97d384f.

Comment 3 Vlad Zahorodnii 2022-12-05 08:48:37 UTC

Update kwindowsystem. You need at least https://invent.kde.org/frameworks/kwindowsystem/-/commit/7cfd7c36eb017242d7a0202db82895be6b8fb81c

Comment 4 Duncan 2022-12-05 08:53:10 UTC

Thanks.  Was just figuring that out and mid-air-ed.  (Gentoo did a qt5 update which forced a kwin rebuild, among others, without doing the preliminaries.)