FS#77816 - [libopenmpt] Unneeded dependency: libogg

Attached to Project: Arch Linux
Opened by Balló György (City-busz) - Saturday, 11 March 2023, 16:40 GMT
Last edited by Toolybird (Toolybird) - Sunday, 12 March 2023, 00:04 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To David Runge (dvzrv)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

It seems libopenmpt is not linked to libogg. Please remove libogg and libogg.so from depends=().
This task depends upon

Closed by  Toolybird (Toolybird)
Sunday, 12 March 2023, 00:04 GMT
Reason for closing:  None
Additional comments about closing:  Reporter says "Handled by this TODO list: https://archlinux.org/todo/libdependslib provides-without-version-specified/&quot ;
Comment by Balló György (City-busz) - Saturday, 11 March 2023, 16:44 GMT
Side note: the makepkg command does not able to fill the version for libogg.so, because it's not linked by this package. Since the libogg.so dependency does not specify the version (=0-64), it makes it ambiguous between libogg and lib32-libogg.
Comment by Toolybird (Toolybird) - Saturday, 11 March 2023, 21:46 GMT
Interesting. The code definitely has `#ifdef MPT_WITH_OGG' sprinkled throughout which suggests a possible indirect dep. Compiling with `--without-ogg' produces a different library so there does appear to be some kind of dep on libogg, just not in the usual library linkage sense. Probably needs more analysis to be certain. Could you please delve into it at a deeper level? At this stage I believe this ticket is suspect.
Comment by Balló György (City-busz) - Saturday, 11 March 2023, 21:53 GMT
It seems the ogg support is present in the package, even though it's not linked to libogg:
$ openmpt123 --credits | grep ogg
Xiph.Org Foundation for libogg
https://xiph.org/ogg/

Might be a header-only dependency. libogg is referenced by the pkg-config file 'libopenmpt.pc', so it seems libogg should be kept as a dependency, but 'libogg.so' should be removed.

Loading...