Merge pull request #7146 from xmake-io/runtimes

Fix runtimes

3 files changed, 58 insertions, 6 deletions

diff --git a/xmake/rules/c++/config/main.lua b/xmake/rules/c++/config/main.lua
index 68bf5a064..6dbc160e7 100644
--- a/xmake/rules/c++/config/main.lua
+++ b/xmake/rules/c++/config/main.lua
@@ -23,6 +23,7 @@ import("rules.c++.config.basic", {rootdir = os.programdir(), alias = "config_bas
 import("rules.c++.config.dynamic_debugging", {rootdir = os.programdir(), alias = "config_dynamic_debugging"})
 import("rules.c++.config.optimization", {rootdir = os.programdir(), alias = "config_optimization"})
 import("rules.c++.config.sanitizer", {rootdir = os.programdir(), alias = "config_sanitizer"})
+import("rules.c++.config.runtime", {rootdir = os.programdir(), alias = "config_runtime"})
 
 -- main entry
 function main(target, sourcekind)
@@ -36,6 +37,9 @@ function main(target, sourcekind)
     -- config optimization configs
     config_optimization(target, sourcekind)
 
+    -- config runtime configs
+    config_runtime(target)
+
     -- config sanitizer configs
     config_sanitizer(target, sourcekind)
 end
diff --git a/xmake/rules/c++/config/runtime.lua b/xmake/rules/c++/config/runtime.lua
new file mode 100644
index 000000000..9161e92eb
--- /dev/null
+++ b/xmake/rules/c++/config/runtime.lua
@@ -0,0 +1,54 @@
+--!A cross-platform build utility based on Lua
+--
+-- Licensed under the Apache License, Version 2.0 (the "License");
+-- you may not use this file except in compliance with the License.
+-- You may obtain a copy of the License at
+--
+--     http://www.apache.org/licenses/LICENSE-2.0
+--
+-- Unless required by applicable law or agreed to in writing, software
+-- distributed under the License is distributed on an "AS IS" BASIS,
+-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+-- See the License for the specific language governing permissions and
+-- limitations under the License.
+--
+-- Copyright (C) 2015-present, Xmake Open Source Community.
+--
+-- @author      ruki
+-- @file        runtime.lua
+--
+
+import("core.project.config")
+
+function main(target)
+    local runtimes = config.get("runtimes")
+    if not runtimes and target:is_plat("windows") then
+        runtimes = config.get("vs_runtime")
+        if runtimes then
+            wprint("--vs_runtime=%s is deprecated, please use --runtimes=%s", runtimes, runtimes)
+        end
+    end
+    if not runtimes and target:is_plat("android") then
+        runtimes = config.get("ndk_cxxstl")
+        if runtimes then
+            wprint("--ndk_cxxstl=%s is deprecated, please use --runtimes=%s", runtimes, runtimes)
+        end
+    end
+    if runtimes and not target:get("runtimes") then
+        if type(runtimes) == "string" then
+            runtimes = runtimes:split(",", {plain = true})
+        end
+        target:set("runtimes", runtimes)
+    end
+
+    -- enable vs runtime as MD by default
+    if target:is_plat("windows") and not target:get("runtimes") then
+        local vs_runtime_default = target:policy("build.c++.msvc.runtime")
+        if vs_runtime_default and target:has_tool("cxx", "cl", "clang", "clangxx", "clang_cl") then
+            if is_mode("debug") then
+                vs_runtime_default = vs_runtime_default .. "d"
+            end
+            target:set("runtimes", vs_runtime_default)
+        end
+    end
+end
diff --git a/xmake/rules/c++/xmake.lua b/xmake/rules/c++/xmake.lua
index dbb24451d..6c16e666a 100644
--- a/xmake/rules/c++/xmake.lua
+++ b/xmake/rules/c++/xmake.lua
@@ -39,9 +39,6 @@ rule("c")
     -- add build rules
     add_deps("c.build")
 
-    -- set compiler runtime, e.g. vs runtime
-    add_deps("utils.compiler.runtime")
-
     -- inherit links and linkdirs of all dependent targets by default
     add_deps("utils.inherit.links")
 
@@ -64,9 +61,6 @@ rule("c++")
     -- add build rules
     add_deps("c++.build")
 
-    -- set compiler runtime, e.g. vs runtime
-    add_deps("utils.compiler.runtime")
-
     -- inherit links and linkdirs of all dependent targets by default
     add_deps("utils.inherit.links")