Skip to content

tests: suppress "built under newer R" warning#7742

Merged
ben-schwen merged 1 commit into
masterfrom
fix_library_warning
May 6, 2026
Merged

tests: suppress "built under newer R" warning#7742
ben-schwen merged 1 commit into
masterfrom
fix_library_warning

Conversation

@aitap
Copy link
Copy Markdown
Member

@aitap aitap commented May 6, 2026

When using the .0 release of R with binary packages from CRAN, library() will complain about the binary package having been built under a newer (but still compatible in this case) version of R. This was already suppressed in most places, but not in test 2265. This previously caused tests mac-rel and win-rel to fail. With the patch they do succeed (mac, win).

Unrelated: the other two failures in lin-dev-clang-cran, lin-dev-gcc-strict-cran were due to makeindex missing from TinyTeX (newly exposed by tools::texi2pdf() in R-devel r89983), now fixed (clang, gcc-strict) in the container images.

Fixes: #7721

When using the .0 release of R with binary packages from CRAN, library()
will complain about the binary package having been built under a newer
(but still compatible in this case) version of R. This was already
suppressed in most places, but not in test 2265.
@aitap aitap requested a review from MichaelChirico as a code owner May 6, 2026 10:28
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.04%. Comparing base (da0d15e) to head (f93567e).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7742   +/-   ##
=======================================
  Coverage   99.04%   99.04%           
=======================================
  Files          87       87           
  Lines       17058    17058           
=======================================
  Hits        16895    16895           
  Misses        163      163           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ben-schwen ben-schwen merged commit 2d9216a into master May 6, 2026
13 checks passed
@ben-schwen ben-schwen deleted the fix_library_warning branch May 6, 2026 11:00
skitsy24 pushed a commit to skitsy24/data.table that referenced this pull request May 13, 2026
When using the .0 release of R with binary packages from CRAN, library()
will complain about the binary package having been built under a newer
(but still compatible in this case) version of R. This was already
suppressed in most places, but not in test 2265.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

R version checks in library() cause warnings that fail tests on GLCI

2 participants