Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 650542 - media-sound/lame-3.100 installs /usr/bin/mlame even with USE=-frontend
Summary: media-sound/lame-3.100 installs /usr/bin/mlame even with USE=-frontend
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-15 03:11 UTC by Mart Raudsepp
Modified: 2018-03-15 08:28 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mart Raudsepp gentoo-dev 2018-03-15 03:11:59 UTC
The ebuild does an unconditional "dobin misc/mlame" in multilib_src_install_all, but that's a shell script that just calls "lame" frontend multiple times or something. So it's rather useless, giving something like this when actually used with USE=-frontend:

/usr/bin/mlame: line 94: mlame_corr: command not found
/usr/bin/mlame: line 94: lame: command not found
converting of 'dummy-message.wav' to 'dummy-message.mp3' failed
/usr/bin/mlame: line 94: mlame_corr: command not found
/usr/bin/mlame: line 94: lame: command not found
converting of 'test.wav' to 'test.mp3' failed


Please consider putting the dobin behind a use frontend check, and check what that mlame_corr command not found is about - sounds like another script from misc/ or somewhere is used by this mlame script, but not installed.
Comment 1 Larry the Git Cow gentoo-dev 2018-03-15 08:28:50 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=de2c52788dc9a6881830eca97da7e3499ea34219

commit de2c52788dc9a6881830eca97da7e3499ea34219
Author:     Lars Wendler <polynomial-c@gentoo.org>
AuthorDate: 2018-03-15 08:27:40 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2018-03-15 08:28:43 +0000

    media-sound/lame: Don't install mlame script
    
    It uses mlame_corr binary that gets never built, only works on little endian
    machines and fails to recognize all wav files.
    Also overhauled the ebuild a bit. Committed straight to stable.
    
    Closes: https://bugs.gentoo.org/650542
    Package-Manager: Portage-2.3.24, Repoman-2.3.6

 .../{lame-3.100.ebuild => lame-3.100-r1.ebuild}    | 26 +++++++++-------------
 1 file changed, 11 insertions(+), 15 deletions(-)