[Bf-blender-cvs] [275ca01] blender-v2.77-release: Fix T47939, take II: Check clang package version, not llvm-devel one.

Bastien Montagne noreply at git.blender.org
Tue Apr 5 10:46:45 CEST 2016


Commit: 275ca01eac78b87d4759b671aecfd1c9ab577a91
Author: Bastien Montagne
Date:   Sat Mar 26 21:07:04 2016 +0100
Branches: blender-v2.77-release
https://developer.blender.org/rB275ca01eac78b87d4759b671aecfd1c9ab577a91

Fix T47939, take II: Check clang package version, not llvm-devel one.

Looks like some distro still provide llvm-3.4-devel, while no more clang-3.4.
Since clang depends on llvm of same version, check clang only should ensure
us we also have matvhing llvm... *sigh*

===================================================================

M	build_files/build_environment/install_deps.sh

===================================================================

diff --git a/build_files/build_environment/install_deps.sh b/build_files/build_environment/install_deps.sh
index 9024ff0..9dde348 100755
--- a/build_files/build_environment/install_deps.sh
+++ b/build_files/build_environment/install_deps.sh
@@ -2642,7 +2642,7 @@ install_DEB() {
     INFO "Forced LLVM building, as requested..."
     _do_compile_llvm=true
   else
-    check_package_DEB llvm-$LLVM_VERSION-dev
+    check_package_DEB clang-$LLVM_VERSION
     if [ $? -eq 0 ]; then
       install_packages_DEB llvm-$LLVM_VERSION-dev clang-$LLVM_VERSION
       have_llvm=true
@@ -3161,21 +3161,28 @@ install_RPM() {
     _do_compile_llvm=true
   else
     # Problem compiling with LLVM 3.2 so match version 3.1 ...
-    check_package_version_match_RPM llvm $LLVM_VERSION
-    if [ $? -eq 0 ]; then
-      if [ "$RPM" = "SUSE" ]; then
+    if [ "$RPM" = "SUSE" ]; then
+      check_package_version_match_RPM llvm-clang-devel $LLVM_VERSION
+      if [ $? -eq 0 ]; then
         install_packages_RPM llvm-devel llvm-clang-devel
+        have_llvm=true
+        LLVM_VERSION_FOUND=$LLVM_VERSION
+        clean_LLVM
       else
-        install_packages_RPM llvm-devel clang-devel
+        # Better to compile it than use minimum version from repo...
+        _do_compile_llvm=true
       fi
-      have_llvm=true
-      LLVM_VERSION_FOUND=$LLVM_VERSION
-      clean_LLVM
     else
-      #
-      # Better to compile it than use minimum version from repo...
-      #
-      _do_compile_llvm=true
+      check_package_version_match_RPM clang-devel $LLVM_VERSION
+      if [ $? -eq 0 ]; then
+        install_packages_RPM llvm-devel clang-devel
+        have_llvm=true
+        LLVM_VERSION_FOUND=$LLVM_VERSION
+        clean_LLVM
+      else
+        # Better to compile it than use minimum version from repo...
+        _do_compile_llvm=true
+      fi
     fi
   fi
 
@@ -3563,11 +3570,11 @@ install_ARCH() {
     INFO "Forced LLVM building, as requested..."
     _do_compile_llvm=true
   else
-    check_package_version_match_ARCH llvm $LLVM_VERSION
+    check_package_version_match_ARCH clang $LLVM_VERSION
     if [ $? -eq 0 ]; then
       install_packages_ARCH llvm clang
       have_llvm=true
-      LLVM_VERSION=`check_package_version_ge_ARCH llvm $LLVM_VERSION_MIN`
+      LLVM_VERSION=`check_package_version_ge_ARCH clang $LLVM_VERSION_MIN`
       LLVM_VERSION_FOUND=$LLVM_VERSION
       clean_LLVM
     else




More information about the Bf-blender-cvs mailing list