Skip to content
This repository has been archived by the owner on Oct 2, 2018. It is now read-only.

Solaris 11 support #99

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

Solaris 11 support #99

wants to merge 4 commits into from

Conversation

ccope
Copy link

@ccope ccope commented Apr 24, 2014

The default recipe and pip_package provider work just fine after adding the package names. I didn't test the source recipe but I suspect it won't work without gmake (GNU make instead of the Sun make).

@ccope
Copy link
Author

ccope commented Sep 2, 2014

Any feedback on this PR?

@coderanger
Copy link
Member

@ccope I can't test this easily so I'm not sure I'm super comfortable supporting it internally.

@ccope
Copy link
Author

ccope commented Sep 2, 2014

Yeah... I put in the work to create a bento packer definition for automatically generating Solaris 11 base boxes, but unfortunately it doesn't seem possible to distribute prebuilt boxes right now :-(

https://github.com/opscode/bento/blob/master/packer/solaris-11-x86.json

@coderanger
Copy link
Member

Even with that, my testing harness is built on docker+travis. So this would have to be manually tested.

@ccope
Copy link
Author

ccope commented Sep 2, 2014

Ah... Well, that's kind of a bummer. Though the cookbook does currently support FreeBSD and SmartOS (a Solaris derivative), which also do not run in Docker containers, I can see the concern of adding more untested code. Given that the scope of the change is so small though, would you consider merging this if I added a disclaimer to the README like this?

The following platforms are not automatically testable on TravisCI. 
If you encounter problems, pull requests with associated tests are welcome.
- FreeBSD
- SmartOS
- Solaris

@coderanger
Copy link
Member

Yeah, thats probably reasonable.

@ccope
Copy link
Author

ccope commented Sep 2, 2014

Hmm. Well, I added the notice. Looks like berkshelf is having some unrelated issues right now which broke the build (failed to download the minitest cookbook).

@ccope ccope force-pushed the master branch 2 times, most recently from 0b15bfe to ea212c3 Compare September 8, 2014 18:39
@ccope
Copy link
Author

ccope commented Sep 8, 2014

@coderanger it looks like any new PR's will fail testing until my last commit gets merged

@ccope ccope force-pushed the master branch 3 times, most recently from b1efb76 to 6716009 Compare February 26, 2015 18:51
@ccope
Copy link
Author

ccope commented Feb 26, 2015

rebased on master, fixed build again

@kuldeepsvirk
Copy link

Virtual machine is best tool to cross platforms.

@coderanger
Copy link
Member

Leaving this open for tracking, this should get moved to a provider subclass in https://github.com/poise/poise-python. Will revisit shortly.

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

Successfully merging this pull request may close these issues.

None yet

3 participants