GH-49738: [R][CI] Re-enable GCC+LTO job once rhub has a GCC 15 image#49795
GH-49738: [R][CI] Re-enable GCC+LTO job once rhub has a GCC 15 image#49795thisisnic wants to merge 5 commits intoapache:mainfrom
Conversation
|
|
|
The error is coming from: arrow/r/tests/testthat/test-dplyr-filter.R Lines 295 to 302 in 36c8c9a Which makes it seem like there's something up with translations / locale not being quite right in the runner / our setup |
|
There were changes in #37811 that never got merged as it got complex, which I'll add in here so hopefully those should help |
| @@ -35,19 +35,25 @@ options(arrow.pull_as_vector = FALSE) | |||
|
|
|||
| with_language <- function(lang, expr) { | |||
There was a problem hiding this comment.
Wait, why do we have our own with_language() here? I might try out the withr one before we merge this...
There was a problem hiding this comment.
Does #37811 (comment) have any info about why we have a separate one?
There was a problem hiding this comment.
Yeah, sort of makes sense.
Rationale for this change
We removed a job as it was too old and didn't reflect CRAN and now we added it in as we have one which does reflect CRAN's environments
What changes are included in this PR?
Add GCC 15 job
Are these changes tested?
On CI, sure
Are there any user-facing changes?
Nah