Bug 477047

Summary: Make QtLocation support optional
Product: [Applications] NeoChat Reporter: Anna <cyber+kde>
Component: GeneralAssignee: Tobias Fella <fella>
Status: RESOLVED DOWNSTREAM    
Severity: normal CC: carl
Priority: NOR    
Version First Reported In: 23.08.3   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
URL: https://bugs.gentoo.org/895516
See Also: https://bugreports.qt.io/browse/QTBUG-116652
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Anna 2023-11-15 13:42:52 UTC
QtLocation 5.15.11 fails to build on Clang 16+ systems.

See:
https://bugreports.qt.io/browse/QTBUG-116652
https://bugs.gentoo.org/895516
Comment 1 Tobias Fella 2023-11-15 14:09:23 UTC
Don't mark your own bug reports as CONFIRMED, please
Comment 2 Tobias Fella 2023-11-15 18:02:17 UTC
I don't think making the dependency optional is a good approach to this problem.

- To be useful, it would have to be done for all apps that use QtLocation, not just NeoChat
- Making a QML dependency optional is unfortunately quite painful
- Disabling something is not a good fix

I'd recommend you either:
- Patch it out in NeoChat's packaging (should be relatively easy, just delete the QML files using QtLocation, remove them from CMake and from whichever qml files start complaining about missing components)
- Patch something in QtLocation - ideally something that works fine; something that just compiles but doesn't work could also be good enough
- Fix LLVM