Fix qbs build when no llvm is present

Some defines were set conditionally, but accessed unconditionally, which
of course breaks the build.
Make sure the defines are always set. Also gather them in a module,
instead of copying the code all over the place.

Change-Id: Iac628da64fb3934872d912510936aeccd0e20247
Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
Reviewed-by: Christian Stenger <christian.stenger@qt.io>

1 files changed, 2 insertions, 10 deletions

diff --git a/src/plugins/projectexplorer/projectexplorer.qbs b/src/plugins/projectexplorer/projectexplorer.qbs
index 89534c13eb..67f07840ac 100644
--- a/src/plugins/projectexplorer/projectexplorer.qbs
+++ b/src/plugins/projectexplorer/projectexplorer.qbs
@@ -14,17 +14,9 @@ Project {
         Depends { name: "Core" }
         Depends { name: "TextEditor" }
         Depends { name: "app_version_header" }
-        Depends { name: "libclang"; required: false }
-
-        Properties {
-            condition: libclang.present
 
-            cpp.defines: {
-                var defines = base;
-                defines.push('CLANG_BINDIR="' + libclang.llvmBinDir + '"');
-                return defines;
-            }
-        }
+        Depends { name: "libclang"; required: false }
+        Depends { name: "clang_defines" }
 
         Group {
             name: "General"