If --disable-gallium is passed, llvm-config isn't checked for, so mark it explicitly as absent, through LLVM_CONFIG=no.
Passing --disable-gallium would result in: | ../configure: line 9739: --version: command not found | ../configure: line 9740: --cppflags: command not found | ../configure: line 9741: --libs: command not found | ../configure: line 9743: --ldflags: command not found With this commit, one gets that instead: | configure: error: LLVM is required to build Gallium R300 on x86 and x86_64 Signed-off-by: Cyril Brulebois <[email protected]> --- configure.ac | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) As a side note: If --disable-gallium --enable-gallium-llvm is supposed to be a supported combination, the whole llvm-config check has to be reworked, and be made independent of $enable_gallium being yes. diff --git a/configure.ac b/configure.ac index 69513c1..5d7288c 100644 --- a/configure.ac +++ b/configure.ac @@ -1459,6 +1459,8 @@ fi if test "x$enable_gallium" = xyes; then SRC_DIRS="$SRC_DIRS gallium gallium/winsys gallium/targets" AC_PATH_PROG([LLVM_CONFIG], [llvm-config], [no]) +else + LLVM_CONFIG=no fi AC_SUBST([LLVM_CFLAGS]) -- 1.7.5.4 _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
