From 3160172b58a728f9bcf940b4b46a65fb40d294f8 Mon Sep 17 00:00:00 2001 From: Nikolai Kosjar Date: Tue, 16 Oct 2018 12:18:24 +0200 Subject: [PATCH] ClangFormat: Fix constructing libs for qbs Assignment after concat() was missing. Change-Id: I9a038cae6372648561a5450e9e3417c4c603ab7e Reviewed-by: Christian Kandeler --- qbs/modules/libclang/functions.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/qbs/modules/libclang/functions.js b/qbs/modules/libclang/functions.js index bdffcc9c23a..594e26d69f5 100644 --- a/qbs/modules/libclang/functions.js +++ b/qbs/modules/libclang/functions.js @@ -107,24 +107,24 @@ function formattingLibs(llvmConfig, qtcFunctions, targetOS) var libs = [] if (qtcFunctions.versionIsAtLeast(clangVersion, MinimumLLVMVersion)) { if (qtcFunctions.versionIsAtLeast(clangVersion, "7.0.0")) { - libs.concat([ + libs.push( "clangFormat", "clangToolingInclusions", "clangToolingCore", "clangRewrite", "clangLex", - "clangBasic", - ]); + "clangBasic" + ); } else { - libs.concat([ + libs.push( "clangFormat", "clangToolingCore", "clangRewrite", "clangLex", - "clangBasic", - ]); + "clangBasic" + ); } - libs.concat(extraLibraries(llvmConfig, targetOS)); + libs = libs.concat(extraLibraries(llvmConfig, targetOS)); } return libs;