Fix build on Windows

Author: learnforpractice

.github/workflows/python-app.yml

@@ -20,12 +20,19 @@ jobs:
     strategy:
       matrix:
         python-version: [3.7, 3.8, 3.9, "3.10"]
-        os: ['windows-latest', 'ubuntu-18.04', 'macos-latest']
+        os: ['windows-2019', 'ubuntu-18.04', 'macos-latest']
         # python-version: [3.9]
         # os: ['windows-latest']
       fail-fast: false
 
     steps:
+      - uses: MinoruSekine/setup-scoop@v1
+        if: ${{ matrix.os == 'windows-2019' }} 
+      - name: Install Dependencies
+        if: ${{ matrix.os == 'windows-2019' }} 
+        shell: bash
+        run: |
+          scoop install ninja gcc
       - name: Install Go
         uses: actions/setup-go@v2
         with:

install.sh

@@ -1,2 +1,2 @@
-python3 -m pip uninstall mixin -y;python3 -m pip install dist/mixin-0.1.0-cp37-cp37*.whl
+python3 -m pip uninstall mixin-python -y;python3 -m pip install dist/mixin_python-*.whl
 

src/CMakeLists.txt

@@ -7,29 +7,35 @@ add_library(_mixin MODULE
 
 if (WIN32)
     add_compile_definitions(MS_WIN64)
+    SET(BUILD_TARGET mixin.lib)
+    SET(BUILD_LIB_CMD go build -o ${CMAKE_CURRENT_SOURCE_DIR}/pyeoskit/mixin.dll -buildmode=c-shared && gendef mixin.dll && lib /def:pyeoskit.def /machine:x64 /out:mixin.lib && cmake -E copy libmixin.bk.h libmixin.h)
+else()
+    SET(BUILD_TARGET libmixin.a)
+    SET(BUILD_LIB_CMD go build -o ${BUILD_TARGET} -buildmode=c-archive)
 endif()
 
 python_extension_module(_mixin)
 
 add_dependencies(_mixin build_mixin)
 
-target_link_libraries(_mixin ${CMAKE_CURRENT_SOURCE_DIR}/mixin/libmixin.a)
+target_link_libraries(_mixin ${CMAKE_CURRENT_SOURCE_DIR}/mixin/${BUILD_TARGET})
 
 target_include_directories(_mixin PRIVATE 
     ${CMAKE_CURRENT_SOURCE_DIR}/mixin
 )
 
-add_custom_command(OUTPUT ${CMAKE_CURRENT_SOURCE_DIR}/mixin/libmixin.a
-    COMMAND go build -o libmixin.a -buildmode=c-archive
+message(STATUS ${CMAKE_CURRENT_SOURCE_DIR}/mixin/${BUILD_TARGET})
+add_custom_command(OUTPUT ${CMAKE_CURRENT_SOURCE_DIR}/mixin/${BUILD_TARGET}
+    COMMAND ${BUILD_LIB_CMD}
     DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/mixin/mixin_api.go
     DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/mixin/main.go
-    COMMENT "Creating libmixin.a"
+    COMMENT "Creating ${BUILD_TARGET}"
     WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/mixin
 )
 
 add_custom_target(build_mixin ALL
-    COMMAND echo "Building libmixin.a"
-    DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/mixin/libmixin.a
+    COMMAND echo "Building ${BUILD_TARGET}"
+    DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/mixin/${BUILD_TARGET}
     # to make quotes printable,for example
     VERBATIM
 )