Bug 377013 – media-libs/mesa should depend on sys-devel/llvm for r300g on amd64/x86

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Matt Turner 2011-07-30 02:34:09 UTC]

r300g depends on llvm, but on x86 and amd64 only.

USE="video_cards_radeon gallium -llvm" will cause configure to fail. Can we maybe bump the relevant Mesa ebuilds to EAPI=4 and use REQUIRED_USE?

[Comment 1 Matt Turner 2011-08-16 19:15:06 UTC]

I've fixed this in mesa-9999.ebuild. The use of the python eclass was unnecessary, since mesa explicitly uses python2, so I removed it, bumped to EAPI=4, and used REQUIRED_USE.

I guess we'll wait another ~6 months for Mesa 7.12 and roll this out for it then.

[Comment 2 Claudio Roberto França Pereira 2011-08-22 14:29:32 UTC]

Isn't there a way to make it optional even if I'm using the radeon driver? I can't think in one, but I'm not so into ebuilds and mesa to be sure.

I mean, I have a R700 card, I don't want to maintain a llvm install to support R300. But I guess that looking into the current kernel .config and search for the R300 firmware would be terribly bad programming.

[Comment 3 Matt Turner 2011-08-22 14:35:28 UTC]

(In reply to comment #2)
> Isn't there a way to make it optional even if I'm using the radeon driver? I
> can't think in one, but I'm not so into ebuilds and mesa to be sure.
> 
> I mean, I have a R700 card, I don't want to maintain a llvm install to support
> R300. But I guess that looking into the current kernel .config and search for
> the R300 firmware would be terribly bad programming.

- if you're using <mesa-9999, there's no dependence on llvm for
  USE=video_cards_radeon

- if you're using =mesa-9999, you can simply compile with
  USE="-video_cards_radeon video_cards_r600" to not build R300 (since r600g 
  doesn't require llvm, I believe)

[Comment 4 Faustus 2011-09-05 05:23:19 UTC]

(In reply to comment #3)
> - if you're using <mesa-9999, there's no dependence on llvm for
>   USE=video_cards_radeon

Not in the ebuild, but there is an actual build-time dependency:

checking for LIBDRM_RADEON... yes
checking for INTEL... yes
checking for INTEL... yes
configure: error: LLVM is required to build Gallium R300 on x86 and x86_64

In addition, I suspect that dependence on LLVM (when necessary) should be in DEPEND and not in RDEPEND.

[Comment 5 Matt Turner 2011-09-05 05:30:34 UTC]

(In reply to comment #4)
> (In reply to comment #3)
> > - if you're using <mesa-9999, there's no dependence on llvm for
> >   USE=video_cards_radeon
> 
> Not in the ebuild, but there is an actual build-time dependency:
> 
> checking for LIBDRM_RADEON... yes
> checking for INTEL... yes
> checking for INTEL... yes
> configure: error: LLVM is required to build Gallium R300 on x86 and x86_64

Yes... that's exactly what this bug is about.

> In addition, I suspect that dependence on LLVM (when necessary) should be in
> DEPEND and not in RDEPEND.

Not sure. Let me know if you confirm this.

[Comment 6 Matt Turner 2011-09-16 04:12:44 UTC]

(In reply to comment #4)
> In addition, I suspect that dependence on LLVM (when necessary) should be in
> DEPEND and not in RDEPEND.

I believe you are correct. I'll fix this in the overlay so that it'll change for Mesa 7.12.

[Comment 7 Matt Turner 2012-02-10 18:42:28 UTC]

I've removed r300's requirement that llvm be used. Committed with mesa-8.0 and mesa-9999.