Merge pull request #7172 from xmake-io/modules

prepare test targets in compile_commands

5 files changed, 73 insertions, 42 deletions

diff --git a/xmake/actions/test/main.lua b/xmake/actions/test/main.lua
index 0e7192d83..85866814a 100644
--- a/xmake/actions/test/main.lua
+++ b/xmake/actions/test/main.lua
@@ -31,6 +31,7 @@ import("private.action.run.runenvs")
 import("private.service.remote_build.action", {alias = "remote_build_action"})
 import("actions.build.main", {rootdir = os.programdir(), alias = "build_action"})
 import("utils.progress")
+import("private.utils.target", {alias = "target_utils"})
 
 -- test target
 function _do_test_target(target, opt)
@@ -522,6 +523,7 @@ function get_tests()
                     if extra.packages then
                         target_new:add("packages", extra.packages)
                     end
+                    target_utils.config_target(target_new)
                     testinfo.target = target_new
                 end
             end
diff --git a/xmake/core/sandbox/modules/import/core/project/project.lua b/xmake/core/sandbox/modules/import/core/project/project.lua
index 8a808e7da..9b69349d0 100644
--- a/xmake/core/sandbox/modules/import/core/project/project.lua
+++ b/xmake/core/sandbox/modules/import/core/project/project.lua
@@ -127,41 +127,10 @@ function sandbox_core_project.check_options()
     end
 end
 
--- config target
-function sandbox_core_project._config_target(target, opt)
-    for _, rule in ipairs(table.wrap(target:orderules())) do
-        local before_config = rule:script("config_before")
-        if before_config then
-            before_config(target, opt)
-        end
-    end
-
-    for _, rule in ipairs(table.wrap(target:orderules())) do
-        local on_config = rule:script("config")
-        if on_config then
-            on_config(target, opt)
-        end
-    end
-    local on_config = target:script("config")
-    if on_config then
-        on_config(target, opt)
-    end
-
-    for _, rule in ipairs(table.wrap(target:orderules())) do
-        local after_config = rule:script("config_after")
-        if after_config then
-            after_config(target, opt)
-        end
-    end
-end
-
+-- config targets
 function sandbox_core_project._config_targets(opt)
-    opt = opt or {}
-    for _, target in ipairs(table.wrap(project.ordertargets())) do
-        if target:is_enabled() then
-            sandbox_core_project._config_target(target, opt)
-        end
-    end
+    import("private.utils.target", {alias = "target_utils"})
+    target_utils.config_targets(opt)
 end
 
 -- config targets
diff --git a/xmake/modules/private/utils/target.lua b/xmake/modules/private/utils/target.lua
index d06f1293c..728bd6016 100644
--- a/xmake/modules/private/utils/target.lua
+++ b/xmake/modules/private/utils/target.lua
@@ -50,6 +50,7 @@ end
 -- @see https://github.com/xmake-io/xmake/issues/3022
 --
 -- e.g.
+--
 -- for all: add_cxxflags("-g")
 -- only for clang: add_cxxflags("clang::-stdlib=libc++")
 -- only for clang and multiple flags: add_cxxflags("-stdlib=libc++", "-DFOO", {tools = "clang"})
@@ -104,6 +105,7 @@ function translate_flags_in_tool(target, flagkind, flags)
     -- @see https://github.com/xmake-io/xmake/issues/3022
     --
     -- e.g.
+    --
     -- for all: add_cxxflags("-g")
     -- only for clang: add_cxxflags("clang::-stdlib=libc++")
     -- only for clang and multiple flags: add_cxxflags("-stdlib=libc++", "-DFOO", {tools = "clang"})
@@ -165,3 +167,40 @@ function check_target_toolchains()
     end
 end
 
+-- config target
+function config_target(target, opt)
+    for _, rule in ipairs(table.wrap(target:orderules())) do
+        local before_config = rule:script("config_before")
+        if before_config then
+            before_config(target, opt)
+        end
+    end
+
+    for _, rule in ipairs(table.wrap(target:orderules())) do
+        local on_config = rule:script("config")
+        if on_config then
+            on_config(target, opt)
+        end
+    end
+    local on_config = target:script("config")
+    if on_config then
+        on_config(target, opt)
+    end
+
+    for _, rule in ipairs(table.wrap(target:orderules())) do
+        local after_config = rule:script("config_after")
+        if after_config then
+            after_config(target, opt)
+        end
+    end
+end
+
+-- config targets
+function config_targets(opt)
+    opt = opt or {}
+    for _, target in ipairs(table.wrap(project.ordertargets())) do
+        if target:is_enabled() then
+            config_target(target, opt)
+        end
+    end
+end
diff --git a/xmake/plugins/project/clang/compile_commands.lua b/xmake/plugins/project/clang/compile_commands.lua
index a017b4a32..4f01354fa 100644
--- a/xmake/plugins/project/clang/compile_commands.lua
+++ b/xmake/plugins/project/clang/compile_commands.lua
@@ -276,6 +276,23 @@ function _add_target(jsonfile, target)
     os.setenvs(oldenvs)
 end
 
+-- get test targets
+-- https://github.com/xmake-io/xmake/issues/4750
+function _get_test_targets()
+    local test_targets = _g.test_targets
+    if test_targets == nil then
+        test_targets = {}
+        for _, test in pairs(test_action.get_tests()) do
+            local target = test.target
+            if not target:is_phony() then
+                table.insert(test_targets, target)
+            end
+        end
+        _g.test_targets = test_targets
+    end
+    return test_targets
+end
+
 -- add targets
 function _add_targets(jsonfile)
     jsonfile:print("[")
@@ -288,15 +305,19 @@ function _add_targets(jsonfile)
         end
     end
     -- https://github.com/xmake-io/xmake/issues/4750
-    for _, test in pairs(test_action.get_tests()) do
-        local target = test.target
-        if not target:is_phony() then
-            _add_target(jsonfile, target)
-        end
+    for _, target in ipairs(_get_test_targets()) do
+        _add_target(jsonfile, target)
     end
     jsonfile:print("]")
 end
 
+-- prepare targets
+function _prepare_targets()
+    target_cmds.prepare_targets()
+    -- https://github.com/xmake-io/xmake/issues/7166
+    target_cmds.prepare_targets(_get_test_targets())
+end
+
 -- generate compilation databases for clang-based tools(compile_commands.json)
 --
 -- references:
@@ -308,7 +329,7 @@ function make(outputdir)
     local oldir = os.cd(os.projectdir())
     local jsonfile = io.open(path.join(outputdir, "compile_commands.json"), "w")
     os.setenv("XMAKE_IN_COMPILE_COMMANDS_PROJECT_GENERATOR", "true")
-    target_cmds.prepare_targets()
+    _prepare_targets()
     _add_targets(jsonfile)
     jsonfile:close()
     os.setenv("XMAKE_IN_COMPILE_COMMANDS_PROJECT_GENERATOR", nil)
diff --git a/xmake/plugins/project/utils/target_cmds.lua b/xmake/plugins/project/utils/target_cmds.lua
index 25413912a..7ea3a13a0 100644
--- a/xmake/plugins/project/utils/target_cmds.lua
+++ b/xmake/plugins/project/utils/target_cmds.lua
@@ -27,8 +27,8 @@ import("private.utils.batchcmds")
 import("private.action.build.target", {alias = "target_buildutils"})
 
 -- prepare targets
-function prepare_targets()
-    local targets_root = target_buildutils.get_root_targets()
+function prepare_targets(targets)
+    local targets_root = targets or target_buildutils.get_root_targets()
     target_buildutils.run_targetjobs(targets_root, {job_kind = "prepare", for_generator = true, jobs = os.default_njob()})
 end