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

Bug 295210

Summary: libdrm should follow VIDEO_CARDS for drm libs compiled
Product: Gentoo Linux Reporter: Aelia <diego.abelenda>
Component: New packagesAssignee: Gentoo X packagers <x11>
Status: RESOLVED DUPLICATE    
Severity: normal CC: ansla80, jer
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 296482    
Attachments: patch to libdrm-2.4.15 that allow to set the drm modules compiled with VIDEO_CARDS

Description Aelia 2009-11-30 16:33:18 UTC
libdrm build intel support by default, even if you don't need it, this is not gentoo-like and not intended on plateforms that doesn't even have pci.

For exemple the compilation crashes on arm because of some operations not defined as atomic.

Reproducible: Always

Steps to Reproduce:
1. emerge libdrm with VIDEO_CARDS="-intel -radeon"

Actual Results:  
files /usr/lib/libdrm_intel.so and /usr/lib/libdrm_radeon.so exist (on platforms that can compile them)

Expected Results:  
they should not exist as I don't want them, and won't use them.

I'll post an easy patch that add support for VIDEO_CARDS in libdrm
Comment 1 Aelia 2009-11-30 16:34:44 UTC
Created attachment 211600 [details, diff]
patch to libdrm-2.4.15 that allow to set the drm modules compiled with VIDEO_CARDS
Comment 2 Chí-Thanh Christopher Nguyễn gentoo-dev 2009-12-01 00:02:13 UTC
Simply changing libdrm is not enough. Also the reverse dependencies such as mesa and xf86-video-* have to be adjusted if one wants to follow this path.
Comment 3 Aelia 2009-12-01 10:11:34 UTC
the change for mesa is only to check if the video_cards_* are the same for libdrm and mesa as mesa already has choice in VIDEO_CARDS, and for xf86-video-* this is only a check if libdrm has video_cards_* for the specific video card. (one line per ebuild, instead of depending only on libdrm should depend on libdrm[video_cards_*=] for mesa and libdrm[video_cards_*] for the xf86_video_*) 

Overall all this is really simple to do, gives better support for other platforms than x86, and gives better control on what is installed or not.
Comment 4 Rémi Cardona (RETIRED) gentoo-dev 2009-12-11 07:40:56 UTC
If you provide a build.log (like you're supposed to do), I'm willing to bet my lunch money this is going to be a dupe of bug #296482.

If not, well, we'll see :)

But do attach the build.log nonetheless.

Thanks

*** This bug has been marked as a duplicate of bug 296482 ***