add back installation of gdal for linux runners

haganjam · Aug 5, 2023 · ee909bb · ee909bb
1 parent 404da45
commit ee909bb
Showing 1 changed file with 17 additions and 12 deletions.
diff --git a/.github/workflows/R-CMD-check.yaml b/.github/workflows/R-CMD-check.yaml
@@ -32,18 +32,23 @@ jobs:
           r-version: ${{ matrix.config.r }}
           http-user-agent: ${{ matrix.config.http-user-agent }}
           use-public-rspm: false
-      # - name: Install OS packages
-      #   run:   |
-      #         if [ "$RUNNER_OS" == "Linux" ]; then
-      #               sudo apt install -y libgdal-dev libudunits2-dev
-      #         elif [ "$RUNNER_OS" == "Windows" ]; then
-      #               echo "$RUNNER_OS not yet supported"
-      #               exit 1
-      #         else
-      #               echo "$RUNNER_OS not yet supported"
-      #               exit 1
-      #         fi
-      #   shell: bash
+      - name: Install OS packages
+      # dependency terra (1.7) requires: C++17, GDAL (>= 2.2.3), GEOS (>= 3.4.0), PROJ (>= 4.9.3), sqlite3
+      # stripped for testing purposes:
+              # elif [ "$RUNNER_OS" == "macOS" ]; then
+              #     brew install pkg-config
+              #     brew install gdal
+              # elif [ "$RUNNER_OS" == "Windows" ]; then
+              #     echo "$RUNNER_OS not yet supported"
+              #     exit 1
+              # else
+              #     echo "$RUNNER_OS not yet supported"
+              #     exit 1
+        run:   |
+              if [ "$RUNNER_OS" == "Linux" ]; then
+                  sudo apt install -y libgdal-dev libudunits2-dev
+              fi
+        shell: bash
       - uses: r-lib/actions/setup-renv@v2
       - uses: r-lib/actions/setup-r-dependencies@v2
         with: