Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 34036 - broken dependency in cdrdao using -oggvorbis
Summary: broken dependency in cdrdao using -oggvorbis
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Martin Holzer (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-21 09:54 UTC by Uwe Hartmann
Modified: 2003-12-30 09:47 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 Uwe Hartmann 2003-11-21 09:54:19 UTC 
cdrdao will install libvorbis-perl and libvorbis if USE='-oggvorbis +perl'

Reproducible: Always
Steps to Reproduce:
1.USE='-oggvorbis perl' emerge -pv cdrdao
2.
3.

Actual Results:  
would compile libvorbis and libvorbis-perl too

Expected Results:  
USE='-oggvorbis +perl' should not require libvorbis-perl and 

patch:
--- /usr/portage/app-cdr/cdrdao/cdrdao-1.1.7-r1.ebuild  2003-11-20 17:36:31.
000000000 +0100
+++ /usr/portage/app-cdr/cdrdao/cdrdao-1.1.7-r1.ebuild    2003-11-21 18:13:03.
000000000 +0100
@@ -16,7 +16,7 @@

 RDEPEND="gnome? ( >=gnome-base/gnome-libs-1.4.1.2-r1
        >=dev-cpp/gnomemm-1.2.2 )
-       x86? ( perl? ( dev-perl/libvorbis-perl ) )
+       x86? ( perl? ( oggvorbis? ( dev-perl/libvorbis-perl ) ) )
        dev-perl/MP3-Info
        dev-perl/Audio-Wav
        dev-perl/Audio-Tools"
Comment 1 Martin Holzer (RETIRED) gentoo-dev 2003-11-21 11:13:24 UTC 
in cvs
Comment 2 Philipp Kern 2003-12-30 09:47:11 UTC 
Here there was oggvorbis in the USE flags, but no perl, so it *did* install the oggvorbis handler, but hadn't listed libvorbis-perl in its dependencies so mp32dao.pl failed.

I'm not sure about the syntax (if it's use <flag1> <flag2> or use <flag1> && use <flag2>), but shouldn't that help?:
--- cdrdao-1.1.7-r1.ebuild~     2003-12-30 18:45:46.456293624 +0100
+++ cdrdao-1.1.7-r1.ebuild      2003-12-30 18:46:11.822437384 +0100
@@ -60,7 +60,7 @@
        insinto /usr/share/cdrdao/mp32dao
        doins contrib/mp32dao/MediaHandler.pm contrib/mp32dao/mp3handler.pm \
                contrib/mp32dao/BaseInfo.pm
-       use oggvorbis && doins contrib/mp32dao/ogghandler.pm
+       use perl oggvorbis && doins contrib/mp32dao/ogghandler.pm
 
        # cdrdao gets definitely installed
        # binary