diff options
| author | Christian Kandeler <christian.kandeler@qt.io> | 2018-01-25 15:55:42 +0100 |
|---|---|---|
| committer | Christian Kandeler <christian.kandeler@qt.io> | 2018-01-25 15:36:24 +0000 |
| commit | 1838057c222a2fa5c8d21f26dc60e8a65f3b486b (patch) | |
| tree | 38ced44d70941910df28ce82589563d2fdb2e0b5 /qbs | |
| parent | a5b82fe827f553bbb4b0ae117f5799fc85cb9a3a (diff) | |
qbs build: Fix clang Probe
Due to the way JavaScript's split() function works, an empty string
could end up in the list of dynamic libraries, causing a warning when
building the pchmanagerbackend.
Change-Id: I5af834d4f98af379ee346556b47c57cabf8c98db
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
Diffstat (limited to 'qbs')
| -rw-r--r-- | qbs/modules/libclang/functions.js | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/qbs/modules/libclang/functions.js b/qbs/modules/libclang/functions.js index 29d093dcad..7d5e4149fc 100644 --- a/qbs/modules/libclang/functions.js +++ b/qbs/modules/libclang/functions.js @@ -16,7 +16,10 @@ function readOutput(executable, args) function readListOutput(executable, args) { - return readOutput(executable, args).split(/\s+/); + var list = readOutput(executable, args).split(/\s+/); + if (!list[list.length - 1]) + list.pop(); + return list; } function isSuitableLLVMConfig(llvmConfigCandidate, qtcFunctions) |