From d498a7fcf9bf4dbedd213f9da675094bdf8a80e5 Mon Sep 17 00:00:00 2001 From: Christian Stenger Date: Fri, 17 Aug 2018 10:22:31 +0200 Subject: [PATCH] Clang: Fix splitting flags Correct syntax of using regex substitution. The missing replacement seems to have resulted performing the search case insensitive which in turn resulted in splitting within an option. Fix this by explicitly passing an empty replacement. Change-Id: Ieccde1f9daae445d0ebe3b7454f56d09f0139e3c Reviewed-by: Ivan Donchevskii Reviewed-by: Oswald Buddenhagen Reviewed-by: Orgad Shaneh --- src/shared/clang/clang_installation.pri | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/shared/clang/clang_installation.pri b/src/shared/clang/clang_installation.pri index 6c0ffd96c58..6ed6bcca204 100644 --- a/src/shared/clang/clang_installation.pri +++ b/src/shared/clang/clang_installation.pri @@ -93,9 +93,9 @@ defineReplace(splitFlags) { for (flag, flags_temp) { equals(inside_quotes, 0) { inside_quotes = 1 - flag ~= s,-I\S*, - flag ~= s,/D\S*, - flag ~= s,/Z\S*, + flag ~= s,-I\S*,, + flag ~= s,/D\S*,, + flag ~= s,/Z\S*,, result += $$split(flag, " ") } else { inside_quotes = 0