jonkeane commented on code in PR #46124:
URL: https://github.com/apache/arrow/pull/46124#discussion_r2045083661
##########
docker-compose.yml:
##########
@@ -1745,6 +1746,34 @@ services:
/bin/bash -c "
/arrow/ci/scripts/r_sanitize.sh /arrow"
+ r-clang-ubsan:
+ image: ${REPO}:r-rhub-clang-ubsan-devel-latest
+ build:
+ context: .
+ dockerfile: ci/docker/linux-r.dockerfile
+ cache_from:
+ - ${REPO}:r-rhub-clang-ubsan-devel-latest
+ args:
+ base: rhub/clang-ubsan
+ cmake: ${CMAKE}
+ r_dev: ${ARROW_R_DEV}
+ r_bin: R
+ tz: ${TZ}
+ r_prune_deps: ${R_PRUNE_DEPS}
+ shm_size: *shm-size
+ environment:
+ <<: *common
+ LIBARROW_DOWNLOAD: "false"
+ ARROW_SOURCE_HOME: "/arrow"
+ ARROW_R_DEV: ${ARROW_R_DEV}
+ # To test for CRAN release, delete ^^ these two env vars so we download
the Apache release
Review Comment:
FWIW, I don't think we ever (need) to do this. IMHO I would say let's remove
this + the other sanitizer copy/pasta version of this and we can leave it up
top where it is under the one R docker file.
It was introduced with https://github.com/apache/arrow/pull/6995 and I don't
think we've every really messed with it since. It's good to have an idea of
what one might do to actually run the full real version, but we test it in
other ways in our CI
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
