Bug 380303

Summary:	media-libs/mesa-7.11 ebuild : Compiles ok without sys-libs/llvm
Product:	Gentoo Linux	Reporter:	Helmuth Schmelzer <des-colo>
Component:	[OLD] Library	Assignee:	Matt Turner <mattst88>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bug, des-colo, orlovm, x11
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	383383
Attachments:	mesa-7.11-r1.ebuild no_lvwm_radeon.patch mesa-7.11-r1.ebuild no_llvm.patch dont-require-llvm-for-r300g.patch failed install failed prepare -patch failed

Description Helmuth Schmelzer 2011-08-22 23:53:57 UTC

media-libs/mesa: compile ok, With no_lvwm_radeon.patch
I have added a sentence to the USE llvm, so that when this force is not applied the patch and compiled with llvm, or else apply the patch and compile without llvm in the corrected ebuild

Reproducible: Always

Comment 1 Helmuth Schmelzer 2011-08-22 23:57:00 UTC

Created attachment 284277 [details]
mesa-7.11-r1.ebuild

Ebuild with added no_llvm_radeon.patch possibly, other drivers can be compiled without llvm

Comment 2 Helmuth Schmelzer 2011-08-23 00:02:39 UTC

Created attachment 284281 [details, diff]
no_lvwm_radeon.patch

no_lvwm_radeon.patch for compiling media-libs/mesa without sys-devel/llvm

Comment 3 Helmuth Schmelzer 2011-08-23 00:11:56 UTC

Created attachment 284283 [details]
mesa-7.11-r1.ebuild

Ebuild with added no_llvm.patch. for compilation without sys-libs/llvm

Comment 4 Helmuth Schmelzer 2011-08-23 00:13:44 UTC

Created attachment 284285 [details, diff]
no_llvm.patch

no_llvm.patch for compiling media-libs/mesa without sys-devel/llvm

Comment 5 Matt Turner gentoo-dev

2011-09-16 18:16:57 UTC

What do we think about this, x11@?

Marek wants LLVM to always be installed with r300g, but that's not possible on a lot of platforms, so it's only forced on x86 and amd64.

We can remove this requirement pretty easily. I think SuSE did the same.

Comment 6 Helmuth Schmelzer 2011-09-17 16:23:00 UTC

Yes, Gallium is optional, only in x86 and amd64 archs :)

in others arch, no

Comment 7 Matt Turner gentoo-dev

2011-09-22 18:33:26 UTC

Created attachment 287433 [details, diff]
dont-require-llvm-for-r300g.patch

Sensible patch.

Comment 8 Matt Turner gentoo-dev

2012-02-10 18:41:46 UTC

Added to mesa-8.0 and mesa-9999. Thanks!

Comment 9 David Heidelberg (okias) 2012-05-13 15:30:49 UTC

disabling llvm on x86/x86_64 causing big performance loss, I think where it's possible, should be enabled.

Comment 10 David J Cozatt 2012-07-28 15:34:19 UTC

Recently updated to mesa 8.1-rc($) (Wednesday iirc) and the update choked on the patch for this bug. Commented out the line in src_prepare and it compiled fine on by, Will attach build log. With that update think this patch can be removed. 

penguin4 david # grep r300 /etc/make.conf
VIDEO_CARDS="radeon r300 r600"

Comment 11 David J Cozatt 2012-07-28 15:44:21 UTC

Created attachment 319528 [details]
failed install

Comment 12 David J Cozatt 2012-07-28 16:34:04 UTC

Created attachment 319544 [details]
failed prepare -patch failed

Comment 13 David J Cozatt 2012-07-28 16:36:16 UTC

For attachment in comment 11 see bug #42844

Comment 14 David J Cozatt 2012-07-28 16:37:31 UTC

err Bug #428444