Build on Windows (#12)

* Switch from micromamba action to miniforge

* Only test on python=3.13

* Set up Windows job

* Checkout submodule on Windows

* Back to micromamba, I guess

* Set the FC environment variable

* Use an absolute path to flang

* Use ninja build backend

* Set PREFIX environment variable

* Use contexts for env vars in cmake call

* Make a LINKER variable pointing to the VS linker

* Quote LINKER variable in cmake call

* Checkout bmi-example-fortran v2.1.4

* Restructure for flang 19

* Use build isolation with pip install

* CONDA_PREFIX is now available

* Push MSVC link.exe to front of path

* Remove the link.exe in the Git install

* Quote path to link.exe

* Use dependency function and pkg-config to find deps

* Use LIBRARY_PREFIX env var to install HeatModelF

* Try to best match working example

* Thanks, VS Code

* Append conda libpath to Windows LIB

* Add tests

* Specify path to config file for bmi-tester

* Also set absolute path to bmi-tester root dir

* Don't run bmi-tester on Windows

* Add notes on workflow steps

Author: mdpiper

.github/workflows/test.yml

@@ -6,7 +6,7 @@ env:
   BUILD_DIR: _build
 
 jobs:
-  build-and-test:
+  build-test-unix:
     if:
       github.event_name == 'push' || github.event.pull_request.head.repo.full_name != github.repository
 
@@ -19,7 +19,6 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest]
-        python-version: ["3.11", "3.12", "3.13"]
 
     steps:
       - uses: actions/checkout@v4
@@ -42,13 +41,94 @@ jobs:
         run: |
           make install
 
-      - name: Test BMI
+      - name: Test imports
+        working-directory: ${{ github.workspace }}/examples
         run: |
           python -c 'import pymt_heatf'
+          python -c 'from pymt.models import HeatModelF'
+
+      - name: Test BMI
+        run: |
           make test
 
       - name: Run examples
         working-directory: ${{ github.workspace }}/examples
         run: |
           python heatf_ex.py
           python pymt_heatf_ex.py
+
+  build-test-windows:
+    if:
+      github.event_name == 'push' || github.event.pull_request.head.repo.full_name != github.repository
+
+    runs-on: windows-latest
+
+    env:
+      LIBRARY_PREFIX: $env:CONDA_PREFIX\Library
+
+    defaults:
+      run:
+        shell: pwsh
+
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          submodules: true
+
+      - uses: ilammy/msvc-dev-cmd@v1
+
+      - uses: mamba-org/setup-micromamba@v2
+        with:
+          micromamba-version: latest
+          environment-file: environment.yml
+          init-shell: >-
+            powershell
+
+      # The Fortran conda compiler, flang, needs to be set over the gfortran installed by chocolatety.
+      - name: Set the FC environment variable
+        run: |
+          echo "FC=$env:CONDA_PREFIX\Library\bin\flang-new.exe" | Out-File -FilePath $Env:GITHUB_ENV -Encoding utf8 -Append
+
+      # A link.exe installed by chocolatey was ahead of the MSVC link.exe in the path.
+      - name: Remove link.exe in the Git install
+        run: |
+          Remove-Item -Path "C:\Program Files\Git\usr\bin\link.exe" -Force
+
+      # The flang FortranRuntime.lib wasn't being found. This recommendation comes from the Meson docs.
+      - name: Add conda environment libpath to Windows LIB (not LIBPATH)
+        run: |
+          echo "LIB=$env:LIB;$env:CONDA_PREFIX\Library\lib" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
+
+      - name: View environment variables
+        run: |
+          ls env:
+
+      # The install prefix must be quoted.
+      - name: Configure, build, and install HeatModelF
+        run: |
+          pushd ./external/bmi-example-fortran 
+          cmake -B ${{ env.BUILD_DIR }} -LA -G Ninja -DCMAKE_INSTALL_PREFIX="${{ env.LIBRARY_PREFIX }}" -DCMAKE_BUILD_TYPE=Release
+          cmake --build ${{ env.BUILD_DIR }} --target install --config Release
+          popd
+
+      - name: Build and install package
+        run: |
+          make install
+
+      - name: Test imports
+        working-directory: ${{ github.workspace }}/examples
+        run: |
+          python -c 'import pymt_heatf'
+          python -c 'from pymt.models import HeatModelF'
+
+      # Weird Windows stuff here. If the examples in the next step run to completion, the BMI works.
+      # - name: Test BMI
+      #   working-directory: ${{ github.workspace }}
+      #   run: |
+      #     bmi-test pymt_heatf._bmi:HeatModelF --config-file=${{ github.workspace }}\examples\test.cfg --root-dir=examples -vvv
+
+      - name: Run examples
+        working-directory: ${{ github.workspace }}/examples
+        run: |
+          python heatf_ex.py
+          python pymt_heatf_ex.py

Makefile

@@ -87,4 +87,4 @@ dist: clean ## builds source and wheel package
 	ls -l dist
 
 install: clean ## install the package to the active Python's site-packages
-	python -m pip install . -v
+	python -m pip install . -v --no-build-isolation

external/bmi-example-fortran

@@ -1,7 +1,7 @@
 project(
     'pymt_heatf',
     'c', 'cython', 'fortran',
-    version: '3.1.dev0',
+    version: '3.2.dev0',
     license: 'MIT',
 )
 
@@ -25,10 +25,14 @@ incs = include_directories(
     ]
 )
 
+# gfortran on Linux/macOS can find these libraries with fc.find_library(); flang 19 on Windows cannot.
+bmif_dep = dependency('bmif', method: 'pkg-config')
+heatf_dep = dependency('heatf', method: 'pkg-config')
+bmiheatf_dep = dependency('bmiheatf', method: 'pkg-config')
 deps = [
-    fc.find_library('bmif'),
-    fc.find_library('heatf'),
-    fc.find_library('bmiheatf'),
+    bmif_dep,
+    heatf_dep,
+    bmiheatf_dep,
     py.dependency(),
 ]