improve fetchinfo

3 files changed, 55 insertions, 26 deletions

diff --git a/xmake/core/package/package.lua b/xmake/core/package/package.lua
index 5920dacd4..00ae72768 100644
--- a/xmake/core/package/package.lua
+++ b/xmake/core/package/package.lua
@@ -1846,6 +1846,8 @@ function _instance:_fetch_library(opt)
                     components_base.sysincludedirs = nil
                 end
             end
+            local package_utils = sandbox_module.import("private.utils.package", {anonymous = true})
+            package_utils.fetchinfo_set_concat(fetchinfo)
         end
         if fetchinfo and option.get("verbose") then
             local reponame = self:repo() and self:repo():name() or ""
diff --git a/xmake/modules/lib/detect/find_package.lua b/xmake/modules/lib/detect/find_package.lua
index ac1f78ae1..366a25e88 100644
--- a/xmake/modules/lib/detect/find_package.lua
+++ b/xmake/modules/lib/detect/find_package.lua
@@ -23,30 +23,7 @@ import("core.base.option")
 import("core.project.config")
 import("core.cache.detectcache")
 import("package.manager.find_package")
-
--- concat packages
-function _concat_packages(a, b)
-    local result = table.copy(a)
-    for k, v in pairs(b) do
-        local o = result[k]
-        if o ~= nil then
-            v = table.join(o, v)
-        end
-        result[k] = v
-    end
-    for k, v in pairs(result) do
-        if k == "links" or k == "syslinks" or k == "frameworks" or k == "ldflags" or k == "shflags" then
-            if type(v) == "table" and #v > 1 then
-                -- we need to ensure link orders when removing repeat values
-                v = table.reverse_unique(v)
-            end
-        else
-            v = table.unique(v)
-        end
-        result[k] = v
-    end
-    return result
-end
+import("private.utils.package", {alias = "package_utils"})
 
 -- find package using the package manager
 --
@@ -144,8 +121,8 @@ function main(name, opt)
     end
 
     -- register concat
-    if result and type(result) == "table" then
-        debug.setmetatable(result, {__concat = _concat_packages})
+    if result then
+        package_utils.fetchinfo_set_concat(result)
     end
     return result and result or nil
 end
diff --git a/xmake/modules/private/utils/package.lua b/xmake/modules/private/utils/package.lua
new file mode 100644
index 000000000..72def050a
--- /dev/null
+++ b/xmake/modules/private/utils/package.lua
@@ -0,0 +1,50 @@
+--!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, TBOOX Open Source Group.
+--
+-- @author      ruki
+-- @file        package.lua
+--
+
+-- concat packages
+function _concat_packages(a, b)
+    local result = table.copy(a)
+    for k, v in pairs(b) do
+        local o = result[k]
+        if o ~= nil then
+            v = table.join(o, v)
+        end
+        result[k] = v
+    end
+    for k, v in pairs(result) do
+        if k == "links" or k == "syslinks" or k == "frameworks" or k == "ldflags" or k == "shflags" then
+            if type(v) == "table" and #v > 1 then
+                -- we need to ensure link orders when removing repeat values
+                v = table.reverse_unique(v)
+            end
+        else
+            v = table.unique(v)
+        end
+        result[k] = v
+    end
+    return result
+end
+
+-- set concat for find_package/fetch info
+function fetchinfo_set_concat(fetchinfo)
+    if fetchinfo and type(fetchinfo) == "table" then
+        debug.setmetatable(fetchinfo, {__concat = _concat_packages})
+    end
+end