remove broken env command, add module loads to other steps

NOAA-GFDL · Oct 30, 2023 · 3a28fe4 · 3a28fe4
1 parent cd27b5b
commit 3a28fe4
Showing 1 changed file with 15 additions and 7 deletions.
diff --git a/.github/workflows/parallelworks_autotools_intel.yml b/.github/workflows/parallelworks_autotools_intel.yml
@@ -14,21 +14,29 @@ jobs:
     steps:
     - name: checkout
       uses: actions/checkout@v2
+    # TODO env command doesn't seem to work in the runner
+    # if it was we'd be able not not module load every step
     - name: Set up enviroment
       run: | # writes env variables onto file to allow them to persist across steps
         module load intel/${INTEL_VERSION} impi/${INTEL_VERSION} netcdf/${NC_VERSION} hdf5
         echo "FC=`which mpiifort`"              >> "$GITHUB_ENV"
         echo "CC=`which mpiicc`"                >> "$GITHUB_ENV"
         echo "FCFLAGS=`nf-config --fflags`"     >> "$GITHUB_ENV"
         echo "CFLAGS=`nc-config --cflags`"      >> "$GITHUB_ENV"
-        env >> "$GITHUB_ENV"
-      # env:
-      #  CC: mpifc -fc=${{ matrix.compilers == 'intel-classic' && 'icc' || 'icx'}}
-      #  FC: mpifc -fc=${{ matrix.compilers != 'intel-oneapi' && 'ifort' || 'ifx'}}
+        echo "LDFLAGS=`nc-config --libs`"       >> "$GITHUB_ENV"
+    #    env:
+    #      CC: mpifc -fc=${{ matrix.compilers == 'intel-classic' && 'icc' || 'icx'}}
+    #      FC: mpifc -fc=${{ matrix.compilers != 'intel-oneapi' && 'ifort' || 'ifx'}}
     - name: Configure
-      run: autoreconf -if ./configure.ac && ./configure
+      run: |
+        module load intel/${INTEL_VERSION} impi/${INTEL_VERSION} netcdf/${NC_VERSION} hdf5
+        autoreconf -if ./configure.ac && ./configure
     - name: Compile
-      run: make -j || make
+      run: |
+        module load intel/${INTEL_VERSION} impi/${INTEL_VERSION} netcdf/${NC_VERSION} hdf5
+        make -j || make
     - name: Run test suite
-      run: make check TEST_VERBOSE=1
+      run: |
+        module load intel/${INTEL_VERSION} impi/${INTEL_VERSION} netcdf/${NC_VERSION} hdf5
+        make check TEST_VERBOSE=1