Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 616884 - =media-sound/mumble-1.2.19 should DEPEND on both media-libs/speex and media-libs/speexdsp
Summary: =media-sound/mumble-1.2.19 should DEPEND on both media-libs/speex and media-l...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Lars Wendler (Polynomial-C) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 641504
  Show dependency tree
 
Reported: 2017-04-28 16:15 UTC by Oleh
Modified: 2017-12-17 23:01 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Oleh 2017-04-28 16:15:48 UTC 
As in summary, mumble-1.2.9 will not build if both media-libs/speex or media-libs/speexdsp not installed.

Case 1 (when no speex and speexdsp) present (a snippet):
Reading /var/tmp/portage/media-sound/mumble-1.2.19/work/mumble-1.2.19/celt-0.7.0-build/celt-0.7.0-build.pro
Reading /var/tmp/portage/media-sound/mumble-1.2.19/work/mumble-1.2.19/celt-0.11.0-build/celt-0.11.0-build.pro
Reading /var/tmp/portage/media-sound/mumble-1.2.19/work/mumble-1.2.19/src/mumble/mumble.pro
Project ERROR: Package speex not found

Case 2 (when speex is installed and speexdsp is not installed) (a snippet):

Reading /var/tmp/portage/media-sound/mumble-1.2.19/work/mumble-1.2.19/celt-0.7.0-build/celt-0.7.0-build.pro
Reading /var/tmp/portage/media-sound/mumble-1.2.19/work/mumble-1.2.19/celt-0.11.0-build/celt-0.11.0-build.pro
Reading /var/tmp/portage/media-sound/mumble-1.2.19/work/mumble-1.2.19/src/mumble/mumble.pro
Project ERROR: Package speexdsp not found

Current logic in ebuild is wrong as deps alternate with speex/speedsp with either one accepted.




Reproducible: Always
Comment 1 Oleh 2017-04-28 16:16:54 UTC 
=media-sound/mumble-1.2.19
=media-libs/speex-1.2.0-r1
=media-libs/speexdsp-1.2_rc3
Comment 2 Ben Kohler gentoo-dev 2017-04-28 16:34:51 UTC 
This looks like it may need the same logic that pulseaudio ebuilds have, where it requires older speex, OR newer speex+speexdsp:

    || (
        (
            >=media-libs/speex-1.2.0
            media-libs/speexdsp
        )
        (
            <media-libs/speex-1.2.0
            >=media-libs/speex-1.2_rc1
        )
    )
Comment 3 Andreas Sturmlechner gentoo-dev 2017-12-17 23:01:29 UTC 
This was actually fixed in ad84f658520a105fd6a16c0a9cba49d99d8d0a9c.