Bug 507469 - Error when building kwin-x11 using kde-builder, potentially broken includes
Summary: Error when building kwin-x11 using kde-builder, potentially broken includes
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (other bugs)
Version First Reported In: unspecified
Platform: EndeavourOS Linux
: NOR crash
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-07-25 04:21 UTC by Emil P
Modified: 2025-07-30 22:18 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot of issue. (1.50 MB, image/png)
2025-07-25 04:21 UTC, Emil P
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Emil P 2025-07-25 04:21:07 UTC
Created attachment 183503 [details]
Screenshot of issue.

SUMMARY
After running the following command:
kde-builder workspace

The build fails exactly at kwin-x11. After looking at the logs, I noticed that it is because of compiler errors since some functions are missing.

STEPS TO REPRODUCE
1. Run: kde-builder kwin-x11 -D (The build will fail)
2. Look at the corresponding log file

OBSERVED RESULT
The build failed because functions keyPressed() and keyReleased() in /kwin-x11/src/plugins/kglobalaccel/kglobalaccel_plugin.cpp are not defined

EXPECTED RESULT
The build succeeds, and there are no compiler errors.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: EndeavorOS rolling
KDE Plasma Version: 6.4.3
KDE Frameworks Version: 6.16.0
Qt Version: 5.15.17

ADDITIONAL INFORMATION
input.h is not adjacent to the file; input.h has been moved 2 directories back, and the functions appear to be renamed to checkKeyPressed() and checkKeyReleased() within that input.h
Comment 1 David Edmundson 2025-07-25 15:59:09 UTC
It's passing on CI

update your deps.
Comment 2 Nicolas Fella 2025-07-25 16:00:17 UTC
(In reply to David Edmundson from comment #1)
> It's passing on CI

No, it's not.

See https://invent.kde.org/plasma/kwin-x11/-/merge_requests/54
Comment 3 Emil P 2025-07-30 22:18:31 UTC
Marking as resolved because the associated MR has been merged.