Skip to content
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

Fix solaris install #399

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tduehr
Copy link
Member

@tduehr tduehr commented Jan 8, 2015

No description provided.

orderthruchaos and others added 2 commits January 7, 2015 16:09
Also, allow compiler flag to be set for older/newer versions of Solaris.

Admittedly, it is a bit of a preemtive strike for later versions.
@bahamas10
Copy link

Tested and working on SmartOS

@bixu
Copy link

bixu commented May 15, 2015

I confirm that this is working on the latest LTS release of SmartOS.

@tduehr
Copy link
Member Author

tduehr commented May 15, 2015

Any fis needs to be confirmed on both 10 and 11. I will be testing this shortly.

@bixu
Copy link

bixu commented May 15, 2015

@tduehr Thanks! If you need help testing I can pitch in.

@tduehr
Copy link
Member Author

tduehr commented May 15, 2015

@bixu you have a SPARC machine? I've had problems getting anything to run under emulation for SPARC so realy hardware is needed for testing...

@bixu
Copy link

bixu commented May 15, 2015

I used to, but I no longer own a SPARC box.  I will see if anyone in the community has a zone we can jump on.  In the meantime, can we commit the patch for x86?
Blake

On May 15, 2015, at 05:32 PM, tduehr notifications@github.com wrote:

@bixu you have a SPARC machine? I've had problems getting anything to run under emulation for SPARC so realy hardware is needed for testing...

Reply to this email directly or view it on GitHub.

@bixu
Copy link

bixu commented May 15, 2015

@jritorto may have some ideas here...

@tduehr
Copy link
Member Author

tduehr commented May 15, 2015

The SPARC box is more for ongoing testing... This fix should work regardless of the hardware.

@jritorto
Copy link

I have a fresh Solaris 11 install on a t1000 here, I think. Can turn it on
and connect it, give login stuff, etc. if you guys like. However, I'll be
in the garden for the rest of the day here in western Pennsylvania.

On Fri, May 15, 2015 at 11:48 AM, tduehr notifications@github.com wrote:

The SPARC box is more for ongoing testing... This fix should work
regardless of the hardware.


Reply to this email directly or view it on GitHub
#399 (comment).

@bixu
Copy link

bixu commented May 18, 2015

Any update on getting this merged in?

@caius
Copy link

caius commented Apr 28, 2016

Bump, any chance of getting this merged in please? Having to depend on a patch from a pull request to use the ffi gem on SmartOS is annoying 😀

@tduehr
Copy link
Member Author

tduehr commented Apr 28, 2016

We need a patch that works for all versions of solaris. The two proposed each work for different versions and different installs. I haven't had time to build a few VMs for testing yet.

@tduehr
Copy link
Member Author

tduehr commented Apr 28, 2016

@caius can you provide the output of pp RbConfig::CONFIG; nil

@caius
Copy link

caius commented Apr 28, 2016

@tduehr sure thing. Here's the output from ruby 2.1.5 on a SmartOS zone: https://gist.github.com/caius/b0a61d289010cad2dc8427729e35e896

@tduehr
Copy link
Member Author

tduehr commented Jul 12, 2017

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants