-
Notifications
You must be signed in to change notification settings - Fork 217
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possible breaking changes to macos-latest
runner for older versions of R
#412
Comments
I don't really see how that particular change could be causing this, we are not relying on the installed R version at all, and the Perhaps something else with the newer environment is causing the issue. |
@jimhester, I looked at this more in detail, working on a fork of @mdneuzerling's repo for a minimal example running a matrix of (macOS, R) combinations for convenience. See https://github.com/riccardoporreca/parity/actions/workflows/metaflow-reprex.yaml?query=branch%3Ametaflow-reprex, and in particular:
By comparing "Where R you?" in the 4 runs, before and after
It looks like the installation of R 4.0 (and similarly for 3.6 I guess) does not explicitly establish a symlink under |
@mdneuzerling, @jimhester, creating the symbolic links for I guess this can be used as the suggested workaround for now in workflows running on Should this be added to the actions/setup-r/src/installer.ts Lines 229 to 242 in a39aa4f
|
* Symlinks to the installed R and Rscript are not established on older R versions with the updated macOS-latest runner, see r-lib/actions#412. * `setup-r` is often stuck forever and runs until the default 6h timeout.
I'm running into this as well in the Tensorflow repo. |
Yes, the reason for this is that the older macOS R installers used to create the symlinks differently, and since Big Sur the old way is not allowed because of Big Sur's tightened security. The newer R installers create the symlinks from a post-install script which works on Big Sur as well. So we'll need to create the symlinks manually, at least on older R versions, exactly as you say above. |
This is now fixed in the |
This issue has been automatically locked. If you believe you have found a related problem, please file a new issue and include a link to this issue |
Hi folks,
I've noticed some builds starting to fail that were previously fine, and I think it may be due to a breaking change being rolled out to the
macos-latest
image. The image is being upgraded to Big Sur starting from September 15th, with the migration intending to be completed by November 3rd. See the relevant issue, which notes that "R was not available for Big Sur when we prepared the image for the first time."We can force a particular OS with
macos-10.15
ormacos-11
. Here's a summary of success/failure with build links:macos-10.15
macos-11
macos-11
macos-11
The job I've got is very simple, and doesn't depend on repository contents. It simply sets up R and runs
Rscript -e '1+1'
.The text was updated successfully, but these errors were encountered: