-
Notifications
You must be signed in to change notification settings - Fork 784
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
Set default MAKE=make on FreeBSD #2263
Conversation
This was set to gmake in rbenv#1381 as a workaround for https://bugs.ruby-lang.org/issues/16331 YJIT builds in Ruby 3.3 previews require the use of BSD make on these platforms, and no supported version of MRI requires the use of gmake, so revert this. JRuby continues to require gmake for jruby-launcher. Fixes Github issue rbenv#2262
Tested on FreeBSD 13.2-RELEASE with 2.7.8, 3.0.4, 3.1.4, 3.2.2, 3.3.0-preview2, and jruby-9.4.3.0. |
Where is a Lines 738 to 757 in 27a5309
Is it during the |
Yep.
|
Interesting, do you know why gmake doesn't work in that case? |
There are only Linux and Darwin paths for linking on the GNU side. The non-GNU rules appear to be equivalent to the Linux path so I'm not sure why the author chose to put in the comment instead of extending the conditional. Doing that certainly appears to work... |
I submitted a fix upstream. |
This was set to gmake in #1381 as a workaround for https://bugs.ruby-lang.org/issues/16331
YJIT builds in Ruby 3.3 previews require the use of BSD make on these platforms, and no supported version of MRI requires the use of gmake, so revert this.
JRuby continues to require gmake for jruby-launcher.