This is an automated email from the ASF dual-hosted git repository.
jiayu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/sedona-db.git
The following commit(s) were added to refs/heads/main by this push:
new b7e90bb chore(r/sedonadb): Add necessary tweaks for Windows CI on the
R package (#117)
b7e90bb is described below
commit b7e90bb885c2bf3592b584e75badfcb54b16972b
Author: Hiroaki Yutani <[email protected]>
AuthorDate: Mon Sep 22 13:00:14 2025 +0900
chore(r/sedonadb): Add necessary tweaks for Windows CI on the R package
(#117)
---
.github/workflows/r.yml | 24 +++++++++++++++---------
c/sedona-proj/README.md | 2 +-
c/sedona-tg/README.md | 2 +-
3 files changed, 17 insertions(+), 11 deletions(-)
diff --git a/.github/workflows/r.yml b/.github/workflows/r.yml
index 3fdf161..f9ef4d2 100644
--- a/.github/workflows/r.yml
+++ b/.github/workflows/r.yml
@@ -36,6 +36,9 @@ jobs:
test:
runs-on: ${{ matrix.config.os }}
name: ${{ matrix.config.os }} (${{ matrix.config.r }})
+ defaults:
+ run:
+ shell: bash
strategy:
fail-fast: false
@@ -43,6 +46,7 @@ jobs:
config:
- {os: ubuntu-latest, r: 'release'}
- {os: macos-latest, r: 'release'}
+ # - {os: windows-latest, r: 'release'}
env:
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
@@ -63,6 +67,12 @@ jobs:
rustup toolchain install stable --no-self-update
rustup default stable
+ # GHA runner installs GNU target by default, but --no-self-update might
+ # remove it, so run `target add` to ensure the GNU target installed.
+ - name: Setup Rust (Windows)
+ if: matrix.config.os == 'windows-latest'
+ run: rustup target add x86_64-pc-windows-gnu
+
- name: Install dependencies (Linux)
if: matrix.config.os == 'ubuntu-latest'
run: sudo apt-get update && sudo apt-get install -y libgeos-dev
@@ -76,19 +86,15 @@ jobs:
# Update this key to force a new cache
prefix-key: "r-v1"
- - name: Install build dependencies
- run: |
- R -e 'install.packages(c("nanoarrow", "geoarrow"))'
-
- - name: Install R Package
- run: |
- R CMD INSTALL r/sedonadb --preclean
-
- uses: r-lib/actions/setup-r-dependencies@v2
with:
needs: check
working-directory: r/sedonadb
+ - name: Install R Package
+ run: |
+ R CMD INSTALL r/sedonadb --preclean
+
- name: Test R Package
run: |
- R -e 'library(testthat); test_local("r/sedonadb",
MultiReporter$new(list(CheckReporter$new(), LocationReporter$new())))'
+ R -e 'library(testthat); test_local("r/sedonadb",
MultiReporter$new(list(CheckReporter$new(), LocationReporter$new())))'
diff --git a/c/sedona-proj/README.md b/c/sedona-proj/README.md
index 76c896c..1c9312d 100644
--- a/c/sedona-proj/README.md
+++ b/c/sedona-proj/README.md
@@ -35,7 +35,7 @@ to add it manually (hopefully, this will be automated).
### Windows
On Windows, you might need to set `PATH` and `PKG_CONFIG_SYSROOT_DIR` for
-`pkg-config`. For example, when using Rtools45, you progbably need these
+`pkg-config`. For example, when using Rtools45, you probably need these
envvars:
```ps1
diff --git a/c/sedona-tg/README.md b/c/sedona-tg/README.md
index 2e0e5b1..d0e1942 100644
--- a/c/sedona-tg/README.md
+++ b/c/sedona-tg/README.md
@@ -35,7 +35,7 @@ to add it manually (hopefully, this will be automated).
### Windows
On Windows, you might need to set `PATH` and `PKG_CONFIG_SYSROOT_DIR` for
-`pkg-config`. For example, when using Rtools45, you progbably need these
+`pkg-config`. For example, when using Rtools45, you probably need these
envvars:
```ps1
