Bug 590418

Summary:	media-plugins/audacious-plugins-3.7.1: incorrect media-libs/adplug dependency
Product:	Gentoo Linux	Reporter:	Gino McCarty <onigino>
Component:	Current packages	Assignee:	Tony Vroon (RETIRED) <chainsaw>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	sound
Priority:	Normal
Version:	unspecified
Hardware:	AMD64
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log emerge --info

Description Gino McCarty 2016-08-03 20:15:33 UTC

Audacious-plugins has its own build in version of adplug
It does not use adplug libraries ever
also the configure --enable-adplug is ignored (with error in build log)

This dependency should be removed 

PS: modplug should really be optional.. its 2016 guys..


Reproducible: Always

Steps to Reproduce:
grey@greyslug /tmp/audacious-plugins-3.7.1 $ ./configure --enable-adplug
configure: WARNING: unrecognized options: --enable-adplug

Also from configure.ac its clearly hardcoded to yes always
---SNIP----
grey@greyslug /tmp/audacious-plugins-3.7.1 $ more configure.ac|grep adplug
INPUT_PLUGINS="adplug metronom psf tonegen vtx xsf"
echo "  AdLib synthesizer (adplug):             yes"
---SNIP---

Comment 1 Coacher 2016-08-06 09:37:27 UTC

(In reply to Harvey Mittens from comment #0)
> PS: modplug should really be optional.. its 2016 guys..
Please file a separate bugreport for this.

> It does not use adplug libraries ever
> also the configure --enable-adplug is ignored (with error in build log)
> ...
> Steps to Reproduce:
> grey@greyslug /tmp/audacious-plugins-3.7.1 $ ./configure --enable-adplug
> configure: WARNING: unrecognized options: --enable-adplug
Please attach complete build.log that demonstrates this problem
and the output of 'emerge --info media-plugins/audacious-plugins'.

Comment 2 BT 2016-10-04 10:52:30 UTC

Created attachment 449116 [details]
build.log

Comment 3 BT 2016-10-04 10:52:49 UTC

Created attachment 449118 [details]
emerge --info

Comment 4 BT 2016-10-04 11:03:51 UTC

The media-plugins/audacious-plugins ebuild has multiple QA issues and the bug summary should be changed to reflect that.

In addition to the unrecognized adplug configure option, the statusicon configure option is also unrecognized.

configure: WARNING: unrecognized options: --enable-statusicon, --disable-adplug

media-libs/libmodplug support is optional, and an appropriate USE flag should be added.

Comment 5 Enne Eziarc 2017-08-02 23:18:25 UTC

With audacious-plugins-3.8.2 media-libs/adplug is now pulled in as a dependency of USE=qt5, and USE=gtk3, and is also a hardcoded dep regardless? Why does the ebuild have IUSE=adplug at all then?

Comment 6 David Seifert gentoo-dev

2017-11-05 11:31:58 UTC


*** This bug has been marked as a duplicate of bug 572796 ***