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

Bug 317639

Summary: x11-libs/libdrm: [PATCH] allow building only {intel,nouveau,radeon,vmware} support
Product: Gentoo Linux Reporter: Matt Turner <mattst88>
Component: New packagesAssignee: Gentoo X packagers <x11>
Status: RESOLVED FIXED    
Severity: normal CC: esigra
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: patch to allow libdrm to be built with only {intel,nouveau,radeon,vmware} support

Description Matt Turner gentoo-dev 2010-04-28 15:00:59 UTC
Attached patch allows libdrm to be built with only the video_cards_* you care about.

It won't break anything, since users will already have the appropriate video card variable set for other packages like mesa and xorg-drivers.

It also fixes braindead bugs like 297630 since video_cards_intel should already be masked on ARM, and partially fixes other silliness like bug 296482 by allowing platforms where libdrm doesn't matter to only compile core libdrm and not all the device-dependent components.

I'm not sure if my IUSE= usage is correct, so please check that. Also, libkms is only supported on intel, nouveau, and vmware, so only allow it to be built in conjunction with one of them.
Comment 1 Matt Turner gentoo-dev 2010-04-28 15:01:31 UTC
Created attachment 229527 [details, diff]
patch to allow libdrm to be built with only {intel,nouveau,radeon,vmware} support
Comment 2 Rémi Cardona (RETIRED) gentoo-dev 2010-04-29 06:45:06 UTC
Yeah, we might have to do something like this now...

Cheers
Comment 3 Chí-Thanh Christopher Nguyễn gentoo-dev 2010-04-30 14:49:50 UTC
Doing this would also require to add USE dependencies to reverse deps (mesa, xf86-video-*)
Comment 4 Matt Turner gentoo-dev 2010-04-30 21:58:08 UTC
(In reply to comment #3)
> Doing this would also require to add USE dependencies to reverse deps (mesa,
> xf86-video-*)

Yes, good observation.

I'd suggest that we fix this bug at the same time as bug 308497, since it will require the same sort of changes to xf86-video-{ati,intel}.

Thanks. Let me know how I should proceed.
Comment 5 Chí-Thanh Christopher Nguyễn gentoo-dev 2010-09-30 03:49:25 UTC
Fixed in CVS.