Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 317639 - x11-libs/libdrm: [PATCH] allow building only {intel,nouveau,radeon,vmware} support
Summary: x11-libs/libdrm: [PATCH] allow building only {intel,nouveau,radeon,vmware} su...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-28 15:00 UTC by Matt Turner
Modified: 2010-09-30 03:49 UTC (History)
1 user (show)

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


Attachments
patch to allow libdrm to be built with only {intel,nouveau,radeon,vmware} support (libdrm-9999.patch,1.23 KB, patch)
2010-04-28 15:01 UTC, Matt Turner
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.