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
#360: Fix solaris 10 match so it doesn't match solaris 11 #391
base: master
Are you sure you want to change the base?
Conversation
'solaris2.11' =~ /solaris(!?2\.11)/ will return 0, which in ruby is true, not false so std=c99 is still showing up on solaris 11.
Any update on this pull request? This also effects installing ffi on SmartOS. |
+1 to fixing this :) |
This pull request overlaps with #382, fixing the same issue. Can we get consensus on which one should be merged? |
Please try #399 as a fix for this. |
I will try this fix on SmartOS as soon as I can. Sent from my iPhone
|
Tested and working on SmartOS |
+1 to fixing this - what can I do to help? |
bump 😃 |
Would pulling this flag from RbConfig work? I don't have a Solaris box to test this properly yet. What do I need to install in a VM to replicate your environment? |
'solaris2.11' =~ /solaris(!?2.11)/ will return 0, which in ruby is true, not false so std=c99 is still showing up on solaris 11.