Summary: | cannot build mesa 9.2 without llvm | ||
---|---|---|---|
Product: | Mesa | Reporter: | Xavier Miller <xavier.miller> |
Component: | Drivers/Gallium/i915g | Assignee: | Matt Turner <mattst88> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | dri-devel, email |
Version: | 9.2 | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | Fix order of "and"/"or" for AM_CONDITIONAL |
Description
Xavier Miller
2013-08-25 20:10:39 UTC
Created attachment 84625 [details] [review] Fix order of "and"/"or" for AM_CONDITIONAL Seems to be problem within configure.ac, occuring since: http://cgit.freedesktop.org/mesa/mesa/commit/configure.ac?id=0d3b1b0e Added possible patch, needs testing. Seems like test always priorities the _and_ over _or_ :) Kris' patch does the right thing, although it can be simplified to either: test "x$HAVE_GALLIUM_I915" = xyes -o "x$HAVE_GALLIUM_SOFTPIPE" = xyes && test "x$MESA_LLVM" = x1 or test "x$HAVE_GALLIUM_I915" = xyes -a "x$MESA_LLVM" = x1 -o "x$HAVE_GALLIUM_SOFTPIPE" = xyes -a "x$MESA_LLVM" = x1 Feel free to test either one(all three :P) and send the git patch to the ML mesa-dev@lists.freedesktop.org Thanks for spotting this :) The proposed patch worked for me (Gentoo Linux, mesa-9.2.0_rc2) Cleaner patch already in mailing list: http://lists.freedesktop.org/archives/mesa-dev/2013-August/043882.html Ah sorry, I missed that somehow. Committed, thanks! Should get picked to the 9.2 branch before release. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.