Skip to content

GH-48013: [R] Add CI job for musl (Alpine Linux) to replicate CRAN checks #48014

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thisisnic merged 24 commits into from
Dec 17, 2025
Merged

GH-48013: [R] Add CI job for musl (Alpine Linux) to replicate CRAN checks #48014

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
ca514cc
Add Alpine Linux R Dockerfile
thisisnic Nov 3, 2025
786095d
Remove RE2
thisisnic Nov 3, 2025
a9b2fad
Add more docker CI boilerpate
thisisnic Nov 3, 2025
1266ee2
Add stuff to actually install Arrow
thisisnic Nov 3, 2025
1f34820
Add image to compose file
thisisnic Nov 3, 2025
4282113
Add bash
thisisnic Nov 3, 2025
0e3a348
Add alpine-specific commands
thisisnic Nov 29, 2025
22fadf7
Add to system dependencies script too
thisisnic Nov 29, 2025
c45e0f7
add linux headers
thisisnic Nov 29, 2025
ecbbdca
Add python deps
thisisnic Nov 29, 2025
8f2a047
remove python dep
thisisnic Nov 29, 2025
37a7aac
Add not_cran and move to next to other R builds
thisisnic Nov 29, 2025
528de01
pass in not_cran=false and rername
thisisnic Nov 29, 2025
b133606
add texlive dependency
thisisnic Dec 1, 2025
4153585
Add extra tex dependencies
thisisnic Dec 1, 2025
6d465be
add devscripts for checkbashims
thisisnic Dec 1, 2025
bc22b46
Add bash
thisisnic Dec 1, 2025
82c40c0
Remove bashism
thisisnic Dec 1, 2025
cf0c854
Set ARROW_R_DEV to whatever has been passed in
thisisnic Dec 17, 2025
4a3e265
Remove unnecessary NOT_CRAN flag
thisisnic Dec 17, 2025
a63c478
Fix spacing
thisisnic Dec 17, 2025
b4f611c
Don't build vignettes
thisisnic Dec 17, 2025
9a81605
Add missing parentheses
thisisnic Dec 17, 2025
5fa4d16
Revert unnecessary brace and manually set 'NOT_CRAN'
thisisnic Dec 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ci/scripts/r_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ SCRIPT="as_cran <- !identical(tolower(Sys.getenv('NOT_CRAN')), 'true')
build_args <- '--no-build-vignettes'
}

if (!as_cran && requireNamespace('reticulate', quietly = TRUE) && reticulate::py_module_available('pyarrow')) {
if (!as_cran && (requireNamespace('reticulate', quietly = TRUE)) && reticulate::py_module_available('pyarrow')) {
Copy link
Copy Markdown

@bastistician bastistician Dec 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand the added parentheses here, they shouldn't make a difference.

Copy link
Copy Markdown
Member Author

@thisisnic thisisnic Dec 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm...they don't - I tried it locally before/after, and whatever was invoking a Python module install must have succeeded when I ran the "before" version, and so when I ran it again "after", I conflated the change in the environment (and thus no console output) with the code change working.

message('Running flight demo server for tests.')
pid_flight <- sys::exec_background(
'python',
Expand Down
Loading