Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 206641 - media-libs/capseo depens on yasm for all arches
Summary: media-libs/capseo depens on yasm for all arches
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Christian Parpart (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-19 14:18 UTC by Markus Rothe (RETIRED)
Modified: 2008-05-02 15:20 UTC (History)
0 users

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 Markus Rothe (RETIRED) gentoo-dev 2008-01-19 14:18:47 UTC
Hello,

media-libs/capseo depends on yasm for all arches and not only x86 and amd64. yasm should only be needed for x86/amd64.

Please make yasm a conditionally dependency.

Regards,

-corsair
Comment 1 Markus Rothe (RETIRED) gentoo-dev 2008-04-13 06:30:58 UTC
news?
Comment 2 Christian Parpart (RETIRED) gentoo-dev 2008-04-29 09:13:40 UTC
hi,

yeah, thanks for the notice, but one question first: on what arch are you?

thanks so far,
christian.
Comment 3 Christian Parpart (RETIRED) gentoo-dev 2008-04-29 09:17:36 UTC
i checked the ebuild now, and did not see any condition on yasm regarding x86/amd64.

the only conditional code in there, is, that capseo supports accelerated code on x86 and amd64 platforms (thats why it needs yasm for at least those archs there).

What are you refering to?
Comment 4 Markus Rothe (RETIRED) gentoo-dev 2008-04-29 11:31:28 UTC
I'm on ppc64.

I *assumed* it needs yasm on x86/amd64, because it's in DEPEND. I never proved this myself.. /me points to "should" in initial comment. Sorry, for not making this clear enough.
Comment 5 Christian Parpart (RETIRED) gentoo-dev 2008-04-30 08:45:36 UTC
(In reply to comment #4)
> I'm on ppc64.
> 
> I *assumed* it needs yasm on x86/amd64, because it's in DEPEND. I never proved
> this myself.. /me points to "should" in initial comment. Sorry, for not making
> this clear enough.

well, capseo actually may require yasm on x86/amd64 to compile some accelerated code.

so on non-x86/amd64 architechtures, yasm shall not be a dependency though, as capseo doesn't have assembler code for other architechtures (say: ppc/ppc64).

though, i should make yasm a dependency for amd64 and x86 only unless you'll state differently.

do you agree? (before i change)?

christian.
Comment 6 Markus Rothe (RETIRED) gentoo-dev 2008-04-30 19:45:55 UTC
agreed. yasm is only needed on x86/amd64 (I just checked the output on my x86 and amd64 machine).
Comment 7 Christian Parpart (RETIRED) gentoo-dev 2008-05-02 15:20:59 UTC
improved ebuild respectively. thanks though ;)