Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 498812

Summary: media-libs/mesa should RDEPEND on dev-libs/llvm if llvm USE flag is used
Product: Gentoo Linux Reporter: ViliusSutkus89
Component: [OLD] LibraryAssignee: Gentoo X packagers <x11>
Status: RESOLVED FIXED    
Severity: normal CC: nikoli
Priority: Normal Keywords: PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 500368    
Attachments: mesa-9.2.5.ebuild.patch
mesa-10.0.2.ebuild.patch

Description ViliusSutkus89 2014-01-21 17:04:13 UTC
Created attachment 368354 [details, diff]
mesa-9.2.5.ebuild.patch

Currently mesa only DEPENDS (not RDEPENDS) on dev-libs/llvm, this leads to a situation where emerging dev-libs/mesa from BINPKG with llvm USE flag doesn't pull in dev-libs/llvm as a dependency.

Attaching proposed patches.

Also a question:
Does opencl USE flag require llvm as a runtime dep?
Comment 1 ViliusSutkus89 2014-01-21 17:08:01 UTC
Created attachment 368356 [details, diff]
mesa-10.0.2.ebuild.patch
Comment 2 ViliusSutkus89 2014-01-21 17:31:22 UTC
All repeated mentions of "dev-libs/llvm" in the initial comment should be assumed as "sys-devel/llvm". Mental typo.
Comment 3 ViliusSutkus89 2014-01-21 17:35:44 UTC
Tried adding dev-libs/llvm as a RDEPEND to media-libs/mesa-9.1.6, but somewhy emerge --usepkg "=mesa-9.1.6" won't pick it up... have no idea why.
Comment 4 Matt Turner gentoo-dev 2014-02-03 21:51:22 UTC
Oh, wow. I think we probably want to add this to RDEPEND only if llvm-shared-libs.
Comment 5 Chí-Thanh Christopher Nguyễn gentoo-dev 2014-02-24 19:44:00 UTC
Fixed in 9.2.5-r1 and 10.0.3
Comment 6 ViliusSutkus89 2014-02-25 18:05:52 UTC
Thank you.

My deployment script is one line shorter now.