Bug 411237

Summary: Wrong entries in the pkgconfig file for kdb
Product: [Frameworks and Libraries] KDb Reporter: Christophe Marin <christophe>
Component: GeneralAssignee: Jarosław Staniek <staniek>
Status: CLOSED LATER    
Severity: normal    
Priority: NOR    
Version: 3.2.0   
Target Milestone: 3.2.1   
Platform: Other   
OS: All   
Latest Commit: https://invent.kde.org/libraries/kdb/-/commit/4252a53ee0e40dffa34fb507bee17d92e4f1e45e Version Fixed In: 3.2.1
Sentry Crash Report:
Attachments: Example generated KDb3.pc

Description Christophe Marin 2019-08-24 09:12:17 UTC 
The pkgconfig files for kdb, kreport and kproperty contain incorrect entries:

- libdir is hardcoded to /lib
- the files contain non-existent 'Requires' entries

KPropertyWidgets.pc.cmake: KF5WidgetsAddons KF5GuiAddons
KDb.pc.cmake: icu-lo KF5CoreAddons
KReport.pc.cmake: KF5CoreAddons KF5WidgetsAddons KF5GuiAddons
Comment 1 2wxsy58236r3 2019-08-24 11:12:17 UTC 
Have you installed all required devel packages?
Comment 2 Christophe Marin 2019-08-24 11:41:59 UTC 
the required frameworks don't install any pkgconfig file and icu-lo also doesn't exist.
Comment 3 Jarosław Staniek 2019-08-24 20:29:57 UTC 
Propsoed patch would be welcome.

If dependencies do not support pkgconfig then we also can't. This shall be consulted with KDE distro team though if there's someone interested in helping out.
Comment 4 Bug Janitor Service 2023-09-03 21:55:51 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/libraries/kdb/-/merge_requests/13
Comment 5 Jarosław Staniek 2023-09-03 22:07:56 UTC 
Dear Christophe, are you able to test if the generated .pc file updated in the above merge request is useful and makes sense?
How about testing for the KDb 3.2 case, and later for other libraries I maintain. 

Sorry for my ignorance since I am not user of pkg-config: I am not sure if KF5 libraries have correctly support pkg-config, or care about it, are you able to give me some hint? Thanks.

Also attaching the generated file based on the change made in the merge request.
Comment 6 Jarosław Staniek 2023-09-03 22:09:02 UTC 
Created attachment 161388 [details]
Example generated KDb3.pc
Comment 7 Jarosław Staniek 2023-09-05 21:14:11 UTC 
Git commit 4252a53ee0e40dffa34fb507bee17d92e4f1e45e by Jaroslaw Staniek.
Committed on 05/09/2023 at 22:43.
Pushed by krop into branch '3.2'.

Temporarily remove pkg-config support

At least until the situation with dependencies (KF) improves in version 6.
FIXED-IN:3.2.1

M  +0    -1    CMakeLists.txt
D  +0    -12   KDb.pc.cmake
M  +0    -12   cmake/modules/KDbMacros.cmake

https://invent.kde.org/libraries/kdb/-/commit/4252a53ee0e40dffa34fb507bee17d92e4f1e45e