FS#47842 - [lua52] pkg-config file naming inconsistent

Attached to Project: Arch Linux
Opened by Silvan Jegen (Shugyousha) - Thursday, 21 January 2016, 08:47 GMT
Last edited by Anatol Pomozov (anatolik) - Tuesday, 07 June 2016, 04:00 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Anatol Pomozov (anatolik)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

The pkg-config file of the lua51 package is installed as '/usr/lib/pkgconfig/lua5.1.pc' while the .pc file of this package is installed as '/usr/lib/pkgconfig/lua52.pc' (note the missing dot). Because the include paths are '/usr/include/lua5.1' and '/usr/include/lua5.2' respectively, changing this package's .pc file name to '/usr/lib/pkgconfig/lua5.2.pc' makes the naming more consistent and avoids surprising users.
This task depends upon

Closed by  Anatol Pomozov (anatolik)
Tuesday, 07 June 2016, 04:00 GMT
Reason for closing:  Fixed
Comment by Doug Newgard (Scimmia) - Thursday, 21 January 2016, 19:38 GMT
How do other distros name it? I's more important to be consistent there than to be consistent with 5.1.
Comment by Anatol Pomozov (anatolik) - Thursday, 21 January 2016, 20:59 GMT
In Debian it contains both names lua5X.pc and lua5.X.pc

https://packages.debian.org/jessie/amd64/liblua5.1-0-dev/filelist
https://packages.debian.org/sid/amd64/liblua5.2-dev/filelist
https://packages.debian.org/sid/amd64/liblua5.3-dev/filelist

One weird thing I spotted that 5.2 for Jessie has only one *.pc file - lua5.2.pc https://packages.debian.org/jessie/amd64/liblua5.2-dev/filelist
Comment by Silvan Jegen (Shugyousha) - Thursday, 21 January 2016, 22:20 GMT
On openSUSE (Tumbleweed) the pkg-config files for the lua51-devel and lua52-devel packages are being installed as

lua-5.1.pc
lua-5.2.pc

so a different pattern again.
Comment by Anatol Pomozov (anatolik) - Thursday, 21 January 2016, 22:32 GMT
Oh, Debian also contains lua-5.X.pc (i.e. 3 different names)
Comment by Silvan Jegen (Shugyousha) - Friday, 22 January 2016, 07:23 GMT
I can confirm that Ubuntu 15.10 contains all three patterns as well (unsurprisingly).

Adding all those .pc files seems like overkill but it would be the most convenient for the users. I definitely think we should have consistent naming between 5.1 and 5.2 in any case.
Comment by Anatol Pomozov (anatolik) - Monday, 06 June 2016, 22:12 GMT
I just pushed lua51-5.1.5-5 to [testing]. It renames *.pc file to lua51.pc to stay consistent with 5.2 and 5.3

With recent lua updates it is what I have at my computer:

$ ls -l /usr/lib/pkgconfig/lua*
-rw-r--r-- 1 root root 669 Jun 6 15:09 /usr/lib/pkgconfig/lua51.pc
-rw-r--r-- 1 root root 442 May 4 2015 /usr/lib/pkgconfig/lua52.pc
-rw-r--r-- 1 root root 425 Jun 6 15:02 /usr/lib/pkgconfig/lua53.pc
lrwxrwxrwx 1 root root 27 Jun 6 15:02 /usr/lib/pkgconfig/lua.pc -> /usr/lib/pkgconfig/lua53.pc
Comment by Silvan Jegen (Shugyousha) - Tuesday, 07 June 2016, 00:51 GMT
Having these names be consistent across versions is definitely preferred, thanks!

The issue with the different naming patterns between the distros still stands however. If you don't want to make the decision of adding the different .pc-filename-patterns I can close this bug now?
Comment by Anatol Pomozov (anatolik) - Tuesday, 07 June 2016, 00:57 GMT
Different distros use different naming. There is no any consistency here. Only upstream can tell what is "right" naming. I would suggest to raise the question in Lua maillist and ask what is the recommended naming for *.pc files.
Comment by Doug Newgard (Scimmia) - Tuesday, 07 June 2016, 01:18 GMT
There is no recommendation from upstream, they only support it as a static lib built into your project.

Loading...