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

Solaris 11 support - #99

Open
ccope wants to merge 4 commits into
poise:masterfrom
ccope:master
Open

Solaris 11 support#99
ccope wants to merge 4 commits into
poise:masterfrom
ccope:master

Conversation

@ccope

@ccope ccope commented Apr 24, 2014

Copy link
Copy Markdown

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

ccope commented Sep 2, 2014

Copy link
Copy Markdown
Author

Any feedback on this PR?

@coderanger

Copy link
Copy Markdown
Member

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

@ccope

ccope commented Sep 2, 2014

Copy link
Copy Markdown
Author

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
Copy Markdown
Member

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

@ccope

ccope commented Sep 2, 2014

Copy link
Copy Markdown
Author

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
Copy Markdown
Member

Yeah, thats probably reasonable.

@ccope

ccope commented Sep 2, 2014

Copy link
Copy Markdown
Author

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

ccope commented Sep 8, 2014

Copy link
Copy Markdown
Author

@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

ccope commented Feb 26, 2015

Copy link
Copy Markdown
Author

rebased on master, fixed build again

@kuldeepsvirk

Copy link
Copy Markdown

Virtual machine is best tool to cross platforms.

@coderanger

Copy link
Copy Markdown
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

3 participants