Merge pull request #6061 from Freed-Wu/cython

Add rule python.cython

Author: waruqi

tests/projects/python/cython/example/src/example.py

@@ -0,0 +1 @@
+print("Hello, world!")

tests/projects/python/cython/example/xmake.lua

@@ -0,0 +1,7 @@
+add_rules("mode.debug", "mode.release")
+add_requires("python 3.x")
+
+target("example")
+    add_rules("python.cython")
+    add_files("src/*.py")
+    add_packages("python")

tests/projects/pybind/example/src/example.cpp renamed to tests/projects/python/pybind/example/src/example.cpp

@@ -2,7 +2,7 @@ add_rules("mode.release", "mode.debug")
 add_requires("pybind11")
 
 target("example")
-    add_rules("python.library", {soabi = true})
+    add_rules("python.module", {soabi = false})
     add_files("src/*.cpp")
     add_packages("pybind11")
     set_languages("c++11")

tests/projects/pybind/example_with_soabi/xmake.lua renamed to tests/projects/python/pybind/example/xmake.lua

@@ -2,7 +2,7 @@ add_rules("mode.release", "mode.debug")
 add_requires("pybind11")
 
 target("example")
-    add_rules("python.library")
+    add_rules("python.module")
     add_files("src/*.cpp")
     add_packages("pybind11")
     set_languages("c++11")

tests/projects/pybind/example_with_soabi/src/example.cpp renamed to tests/projects/python/pybind/example_with_soabi/src/example.cpp

@@ -0,0 +1,62 @@
+--!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      Wu, Zhenyu
+-- @file        xmake.lua
+--
+
+rule("python.cython")
+    add_deps("python.module")
+    set_extensions(".py", ".pyx")
+
+    on_load(function (target)
+        local language = target:extraconf("rules", "python.cython", "language")
+        if language == "c" then
+            target:add("deps", "c")
+        elseif language == "c++" then
+            target:add("deps", "c++")
+        end
+    end)
+
+    before_buildcmd_file(function (target, batchcmds, sourcefile, opt)
+        import("lib.detect.find_tool")
+
+        local cython = assert(find_tool("cython"), "cython not found! please `pip install cython`.")
+        local language = target:extraconf("rules", "python.cython", "language")
+        local ext = "c"
+        local arg = "-3"
+        if language == "c++" then
+            ext = "cc"
+            arg = arg .. "+"
+        end
+        local dirname = path.join(target:autogendir(), "rules", "python", "cython")
+        local sourcefile_c = path.join(dirname, path.basename(sourcefile) .. "." .. ext)
+
+        -- add objectfile
+        local objectfile = target:objectfile(sourcefile_c)
+        table.insert(target:objectfiles(), objectfile)
+
+        -- add commands
+        batchcmds:show_progress(opt.progress, "${color.build.object}compiling.python %s", sourcefile)
+        batchcmds:mkdir(path.directory(sourcefile_c))
+        batchcmds:vrunv(cython.program, {arg, "-o", path(sourcefile_c), path(sourcefile)})
+        batchcmds:compile(sourcefile_c, objectfile)
+
+        -- add deps
+        batchcmds:add_depfiles(sourcefile)
+        batchcmds:set_depmtime(os.mtime(objectfile))
+        batchcmds:set_depcache(target:dependfile(objectfile))
+    end)

tests/projects/pybind/example/xmake.lua renamed to tests/projects/python/pybind/example_with_soabi/xmake.lua

@@ -0,0 +1,25 @@
+--!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        xmake.lua
+--
+
+rule("python.library")
+    on_config(function (target)
+        wprint('deprecated: please use add_rules("python.module") instead of add_rules("python.library")')
+    end)
+    add_deps("python.module")

xmake/rules/python/cython/xmake.lua

@@ -19,13 +19,13 @@
 --
 
 -- @see https://github.com/xmake-io/xmake/issues/1896
-rule("python.library")
+rule("python.module")
     on_config(function (target)
         target:set("kind", "shared")
         target:set("prefixname", "")
         target:add("runenvs", "PYTHONPATH", target:targetdir())
-        local soabi = target:extraconf("rules", "python.library", "soabi")
-        if soabi then
+        local soabi = target:extraconf("rules", "python.module", "soabi")
+        if soabi == nil or soabi then
             import("lib.detect.find_tool")
             local python = assert(find_tool("python3"), "python not found!")
             local result = try { function() return os.iorunv(python.program, {"-c", "import sysconfig; print(sysconfig.get_config_var('EXT_SUFFIX'))"}) end}