First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 34036
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Martin Holzer (RETIRED) <mholzer@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Uwe Hartmann <u.hartl@gmx.de>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 34036 depends on: Show dependency tree
Show dependency graph
Bug 34036 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2003-11-21 09:54 0000
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 From Martin Holzer (RETIRED) 2003-11-21 11:13:24 0000 -------
in cvs

------- Comment #2 From Philipp Kern 2003-12-30 09:47:11 0000 -------
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

First Last Prev Next    No search results available      Search page      Enter new bug