Fix python build with Windows (#648)

IB-8304

Signed-off-by: Raul Metsma <[email protected]>

.github/workflows/build.yml

@@ -22,7 +22,6 @@ jobs:
       run: |
         brew update
         brew install --formula ninja swig doxygen boost
-        brew unlink [email protected] || true
         brew unlink [email protected] || true
         brew unlink [email protected] || true
         brew unlink openssl@3 || true
@@ -136,8 +135,10 @@ jobs:
         include:
         - toolset: 143
           image: windows-2022
+          vcvars: "C:\\Program Files\\Microsoft Visual Studio\\2022\\Enterprise\\VC\\Auxiliary\\Build\\vcvarsall.bat"
         - toolset: 142
           image: windows-2019
+          vcvars: "C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Enterprise\\VC\\Auxiliary\\Build\\vcvarsall.bat"
         - platform: x86
           setenv: amd64_x86
         - platform: x64
@@ -166,17 +167,13 @@ jobs:
         Invoke-WebRequest -UserAgent "Wget" "https://sourceforge.net/projects/swig/files/swigwin/swigwin-4.2.1/swigwin-4.2.1.zip/download" -OutFile swig.zip
         tar xf swig.zip
     - uses: actions/setup-python@v5
-      if: matrix.platform == 'x86'
+      if: matrix.platform != 'arm64'
       with:
         python-version: 3.12
-        architecture: x86
-    - name: Setup dev env
-      uses: ilammy/msvc-dev-cmd@v1
-      with:
-        arch: ${{ matrix.setenv }}
+        architecture: ${{ matrix.platform }}
     - name: Build
       run: |
-        & .\build.ps1 `
+        & "${{ matrix.vcvars }}" ${{ matrix.setenv }} "&&" pwsh build.ps1 `
           -swig ${{ github.workspace }}/swigwin-4.2.1/swig.exe `
           -doxygen "C:/Program files/doxygen/bin/doxygen.exe" `
           -boost

CMakeLists.txt

@@ -77,7 +77,7 @@ endif()
 find_package(SWIG)
 if(SWIG_FOUND)
     find_package(JNI)
-    find_package(Python3 COMPONENTS Interpreter Development)
+    find_package(Python3 COMPONENTS Development)
     if((WIN32 OR APPLE) AND CMAKE_VERSION VERSION_GREATER_EQUAL "3.24")
         find_package(Python3 COMPONENTS Development.SABIModule)
     endif()

build.ps1

@@ -43,13 +43,11 @@ if($doxygen) {
   $cmakeext += "-DDOXYGEN_EXECUTABLE=$doxygen"
   $wixext += "-d", "docLocation=$(Get-Location)/$platform/share/doc/libdigidocpp"
 }
-if($env:VSCMD_ARG_HOST_ARCH -ne "arm64") {
+if($platform -eq "arm64" -and $env:VSCMD_ARG_HOST_ARCH -ne "arm64") {
   $cmakeext += "-DCMAKE_DISABLE_FIND_PACKAGE_Python3=yes"
   $wixext += "-d", "disablePython=1"
-  if($platform -eq "arm64") {
-    $boost = $false
-  }
-} 
+  $boost = $false
+}
 if($boost) {
   $cmakeext += "-DVCPKG_MANIFEST_FEATURES=tests"
   $target += "check"

libdigidocpp.wxs

@@ -105,12 +105,6 @@
       <Files Include="$(var.libdigidocpp)\include\**" />
     </ComponentGroup>
 
-<?ifdef var.docLocation ?>
-    <ComponentGroup Id="Documentation" Directory="INSTALLFOLDER" Subdirectory="documentation">
-      <Files Include="$(var.docLocation)\**" />
-    </ComponentGroup>
-<?endif?>
-
     <Feature Id="InstallLibdigidocpp" Level="1" Title="Libdigidocpp" Display="expand" ConfigurableDirectory="INSTALLFOLDER">
       <ComponentGroupRef Id="Runtime" />
       <ComponentGroupRef Id="Dependencies" />
@@ -124,7 +118,7 @@
       </Feature>
 <?ifdef var.docLocation ?>
       <Feature Id="InstallDocumentation" Level="1" Title="Documentation">
-        <ComponentGroupRef Id="Documentation" />
+        <Files Include="$(var.docLocation)\**" Directory="INSTALLFOLDER" Subdirectory="documentation" />
       </Feature>
 <?endif?>
     </Feature>

src/CMakeLists.txt

@@ -207,15 +207,11 @@ if(SWIG_FOUND)
         endif()
         #configure_file(setup.py.cmake setup.py)
         #install(CODE "execute_process(COMMAND python3 ${CMAKE_CURRENT_BINARY_DIR}/setup.py install)")
-        if(NOT Python3_SITELIB)
-            execute_process(COMMAND /usr/bin/python3 -c
-                "from distutils import sysconfig;print(sysconfig.get_python_lib(plat_specific=False,standard_lib=False))"
-                OUTPUT_VARIABLE Python3_SITELIB OUTPUT_STRIP_TRAILING_WHITESPACE)
-        endif()
         if(WIN32)
             set(Python3_SITELIB ${CMAKE_INSTALL_BINDIR})
             install(FILES $<TARGET_PDB_FILE:digidoc_python> DESTINATION ${Python3_SITELIB} OPTIONAL)
         else()
+            find_package(Python3 COMPONENTS Interpreter) # for Python3_SITELIB
             set_target_properties(digidoc_python PROPERTIES SUFFIX .so)
         endif()
         set_target_properties(digidoc_python PROPERTIES