Bug 411237 - Wrong entries in the pkgconfig file for kdb
Summary: Wrong entries in the pkgconfig file for kdb
Status: CLOSED LATER
Alias: None
Product: KDb
Classification: Frameworks and Libraries
Component: General (show other bugs)
Version: 3.2.0
Platform: Other All
: NOR normal
Target Milestone: 3.2.1
Assignee: Jarosław Staniek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-08-24 09:12 UTC by Christophe Marin
Modified: 2023-09-05 21:45 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 3.2.1


Attachments
Example generated KDb3.pc (388 bytes, text/plain)
2023-09-03 22:09 UTC, Jarosław Staniek
Details

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