Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 32036 - aumix 2.8 dependency & USE inconvenience bugs
Summary: aumix 2.8 dependency & USE inconvenience bugs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-10-26 06:43 UTC by Jonas Berlin
Modified: 2004-02-19 02:10 UTC (History)
2 users (show)

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


Attachments
Fix to only depend on gtk1 when gtk2 is not used. (aumix-gtk.diff,445 bytes, patch)
2003-10-26 12:51 UTC, Doug Weimer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonas Berlin 2003-10-26 06:43:43 UTC
Aumix 2.8 supports both gtk2 and gtk1. But if I select both "gtk" and "gtk2" in USE, it ends up depending on both instead of just gtk2-related libs.

The src_compile does correctly select which gtk to use, but the dependency on gtk1 is unnecessary and adds unnecessary packages at least on my system.

I can also confirm that it works fine (with gtk2) on ppc so I think you can add it to KEYWORDS as stable, or at least unstable.
Comment 1 Doug Weimer 2003-10-26 12:51:45 UTC
Created attachment 19823 [details, diff]
Fix to only depend on gtk1 when gtk2 is not used.

This patch modifies the ebuild to read:

gtk? (
      gtk2? ( >=x11-libs/gtk+-2.0.0 )
      !gtk2? ( =x11-libs/gtk+-1.2* )
)

The format is based on the comments found here:
http://article.gmane.org/gmane.linux.gentoo.devel/13076/match=use+gtk2
Comment 2 SpanKY gentoo-dev 2003-10-27 15:25:37 UTC
why not:
gtk? (
    || (
        gtk2? ( >=x11-libs/gtk+-2.0.0 )
        =x11-libs/gtk+-1.2*
    )
)
or
gtk? (
    gtk2? ( >=x11-libs/gtk+-2.0.0 ) : ( =x11-libs/gtk+-1.2* )
)
Comment 3 Martin Holzer (RETIRED) gentoo-dev 2003-12-18 14:46:26 UTC
gtk QA ?
Comment 4 Spider (RETIRED) gentoo-dev 2003-12-18 15:01:41 UTC
Yep, The patch is correct and reflects the current statement inside the build :
        if use gtk; then
                use gtk2 \
                        && myconf="${myconf} --without-gtk1" \
                        || myconf="${myconf} --without-gtk with-gtk1"
        else
                myconf="${myconf} --without-gtk --without-gtk1";
        fi
, however that looks sorta flawed, shouldn't there be a --with-gtk1 ?  (no -- )



@SpanKY , No. We want to remove all cases of foo? ( bar : baz ) as that syntax was said to be deprecated, and is definitely harder to follow.   Same goes for the || ()  statements.  Don't use more logic than necessary, it makes for complexity which is to be avoided.






Comment 5 Jeremy Huddleston (RETIRED) gentoo-dev 2004-02-19 02:10:26 UTC
looks like it's already in there...