Skip to content

GH-49901: [R] Bump minimum supported R version to 4.2 now that 4.6 is out#49929

Open
thisisnic wants to merge 10 commits intoapache:mainfrom
thisisnic:GH-49901-bump-supported-r
Open

GH-49901: [R] Bump minimum supported R version to 4.2 now that 4.6 is out#49929
thisisnic wants to merge 10 commits intoapache:mainfrom
thisisnic:GH-49901-bump-supported-r

Conversation

@thisisnic
Copy link
Copy Markdown
Member

@thisisnic thisisnic commented May 5, 2026

Rationale for this change

We don't need to support R 4.1 as a new version is out

What changes are included in this PR?

Bump minimum supported version to 4.2

Are these changes tested?

In CI, sure

Are there any user-facing changes?

Nope

Comment thread .github/workflows/r.yml Outdated
run: |
# RTools 40 uses GCC 8.x which does not support C++20 chrono timezones,
# so Arrow uses the vendored date library which requires tzdata
# Download tzdata in case the vendored date library is used for timezones
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Does the RTools bump and associated GCC version increase means that this is no longer needed?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@amoeba would know more than I do, but I think we still need to workaround the tz file issue, but we now do that with an R package instead of the C++ program thing here. Or am I confused about that?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I updated the comment to be clearer now.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we can remove this, but a separate PR is fine. We still need to use this script in other CI jobs where R isn't involved.

Comment thread dev/tasks/macros.jinja
Comment on lines -280 to -281
{% set r_release = {"ver": "4.2", "rt" : "42"} %}
{% set r_oldrel = {"ver": "4.1", "rt" : "40"} %}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

These variables are never used anywhere

@thisisnic
Copy link
Copy Markdown
Member Author

@github-actions crossbow submit -g r

@github-actions github-actions Bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels May 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Revision: 682fe0a

Submitted crossbow builds: ursacomputing/crossbow @ actions-8480702cfb

Task Status
r-binary-packages GitHub Actions
r-recheck-most GitHub Actions
test-r-alpine-linux-cran GitHub Actions
test-r-arrow-backwards-compatibility GitHub Actions
test-r-depsource-system GitHub Actions
test-r-dev-duckdb GitHub Actions
test-r-devdocs GitHub Actions
test-r-extra-packages GitHub Actions
test-r-fedora-clang GitHub Actions
test-r-gcc-11 GitHub Actions
test-r-gcc-12 GitHub Actions
test-r-install-local GitHub Actions
test-r-install-local-minsizerel GitHub Actions
test-r-linux-as-cran GitHub Actions
test-r-linux-rchk GitHub Actions
test-r-linux-sanitizers GitHub Actions
test-r-linux-valgrind GitHub Actions
test-r-m1-san GitHub Actions
test-r-macos-as-cran GitHub Actions
test-r-offline-maximal GitHub Actions
test-r-ubuntu-22.04 GitHub Actions
test-r-versions GitHub Actions

Comment thread ci/scripts/r_windows_build.sh Outdated
Comment on lines +26 to +36
# Ensure CA certificates are available for pacman (Rtools42+ may not bundle them)
if [ ! -f /usr/ssl/certs/ca-bundle.crt ]; then
mkdir -p /usr/ssl/certs
for src in /c/rtools45/usr/ssl/certs/ca-bundle.crt \
"/c/Program Files/Git/mingw64/etc/ssl/certs/ca-bundle.crt"; do
if [ -f "$src" ]; then
cp "$src" /usr/ssl/certs/ca-bundle.crt
break
fi
done
fi
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Claude addition as job fails if we don't do this - unsure if best approach but looks like a CI-specific failure and will see if it makes the CI pass.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Rtools42 definitely bundles these so something else must be going on. I just checked by installing Rtools42 and /usr/ssl/certs is all set up and pacman works fine.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks for checking, will push some changes which print paths etc to see more of what's happening.

@github-actions github-actions Bot added awaiting change review Awaiting change review awaiting changes Awaiting changes and removed awaiting changes Awaiting changes awaiting change review Awaiting change review labels May 5, 2026
@jonkeane
Copy link
Copy Markdown
Member

jonkeane commented May 6, 2026

Are those two crossbow failures on main too? Do we have issues for them?

@raulcd
Copy link
Copy Markdown
Member

raulcd commented May 6, 2026

Are those two crossbow failures on main too? Do we have issues for them?

Yes, they've been failing on the nightlies for ~a week. See:
https://s3.amazonaws.com/arrow-data/index.html

@thisisnic
Copy link
Copy Markdown
Member Author

The test-r-arrow-backwards-compatibility failure is because there are no binaries available from P3M for those Arrow versions and so we build from source with no Arrow C++ libraries and get failures. Given those versions of Arrow are ancient now, I'll remove the CI job like we did with the older ones in #46294

@thisisnic
Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-arrow-backwards-compatibility

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Revision: ce6d929

Submitted crossbow builds: ursacomputing/crossbow @ actions-756eb23241

Task Status
test-r-arrow-backwards-compatibility GitHub Actions

@thisisnic
Copy link
Copy Markdown
Member Author

thisisnic commented May 6, 2026

The ticket for test-r-macos-as-cran is #49941

@github-actions github-actions Bot added awaiting change review Awaiting change review awaiting changes Awaiting changes and removed awaiting changes Awaiting changes awaiting change review Awaiting change review labels May 6, 2026
@jonkeane
Copy link
Copy Markdown
Member

jonkeane commented May 6, 2026

Is the R / AMD64 Windows C++ RTools 42 ucrt64 failure on main too?

@github-actions github-actions Bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels May 6, 2026
@thisisnic
Copy link
Copy Markdown
Member Author

Is the R / AMD64 Windows C++ RTools 42 ucrt64 failure on main too?

Nope, good spot, this is the one I'm trying to fix with the CA certificate shenanigans that RTools42 dislikes. Will iterate to see if I can sort it.

@github-actions github-actions Bot added awaiting changes Awaiting changes awaiting change review Awaiting change review and removed awaiting change review Awaiting change review awaiting changes Awaiting changes labels May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants