Bug 444650

Summary:	org.kde.syntaxhighlighting is not 'required' despite the changes in 6dab24bc
Product:	[Applications] NeoChat	Reporter:	Fushan Wen <qydwhotmail>
Component:	General	Assignee:	Tobias Fella <fella>
Status:	RESOLVED DOWNSTREAM
Severity:	normal	CC:	carl, christoph
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
See Also:	https://bugzilla.opensuse.org/show_bug.cgi?id=1192170
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Fushan Wen 2021-10-30 07:57:35 UTC

org.kde.syntaxhighlighting should be a required dependency.

https://invent.kde.org/network/neochat/-/commit/6dab24bc7467e58df1d537af35153e9a74d7b674 doesn't fix it.

See also: 
https://bugs.archlinux.org/task/72382
https://bugzilla.opensuse.org/show_bug.cgi?id=1192170

Comment 1 Carl Schwan 2022-08-23 21:50:57 UTC

ecm_find_qmlmodule is the correct way to declare a required runtime dependencies. See https://api.kde.org/ecm/module/ECMFindQmlModule.html

Unfortunately not every distro get it right and don't see the cmake warnings about a missing runtime dependency.

The two linked bugs are also now fixed downstream