nealrichardson commented on issue #38047: URL: https://github.com/apache/arrow/issues/38047#issuecomment-1755635551
I don't think removing the cmake installation is a good idea. I recognize that it's in a legal gray area in terms of CRAN policy, but I don't think it's worth the cost to UX if we remove it. Moreover, assuming CRAN approves the libarrow binary downloading, the source build with cmake is only a fallback anyway for other systems, the "approved" path won't touch it. We don't download by default, we check in several locations and only download if an insufficient version is not found. On CRAN machines, at least the regular check ones I spot-checked, we use system `cmake`, no downloading. You can confirm this in the build logs (go to the check page, click on a check result, and change the URL to be `00install` like https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-debian-gcc/arrow-00install.html). Comparing checksums isn't a bad idea, though IMO I wouldn't do it as part of this release, so as not to draw attention to the cmake code. -- 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]
