jonkeane commented on pull request #11944:
URL: https://github.com/apache/arrow/pull/11944#issuecomment-993571905
Re: `${R_BIN} CMD INSTALL --clean ${INSTALL_ARGS}` That _should_ solve the
problem (though reading
[WRE](https://cran.r-project.org/doc/manuals/r-release/R-exts.html#Configure-and-cleanup)
it was't 100% clear if the `./cleanup` file needed to be up to date with all
the files that needed to be removed (in which case it could still fail if it's
not up to date for some reason) or if `--clean` would do that on it's own). But
more importantly: that would remove the built artifacts from the directory
which might force a re-compile elsewhere. This fix intentionally tried to _not_
impact the shared directory between docker and the host so that if the
developer relies on keeping the artifacts around between builds, those will not
be overwritten or deleted.
--
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]