Bug 477047 - Make QtLocation support optional
Summary: Make QtLocation support optional
Status: RESOLVED DOWNSTREAM
Alias: None
Product: NeoChat
Classification: Applications
Component: General (show other bugs)
Version: 23.08.3
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Tobias Fella
URL: https://bugs.gentoo.org/895516
Keywords:
Depends on:
Blocks:
 
Reported: 2023-11-15 13:42 UTC by Anna
Modified: 2023-11-15 18:02 UTC (History)
1 user (show)

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

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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