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

add solaris support #61

Merged
merged 8 commits into from
Jan 29, 2016
Merged

add solaris support #61

merged 8 commits into from
Jan 29, 2016

Conversation

chris-rock
Copy link
Contributor

  • support for solaris 10 & 11
  • also tested against SmartOS and OmniOS
  • improves platform detection
  • optimize stats handling for Solaris 10

# author: Dominik Richter
# author: Christoph Hartmann

require 'shellwords'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Go shellwords! ...but it's not actually used, is it?

@srenatus
Copy link
Contributor

Sorry for nitpicking 😉

@chris-rock
Copy link
Contributor Author

@srenatus +1 love it

@srenatus
Copy link
Contributor

😀

i.e. anything that doesnt understand `echo -n` for speedup
@arlimus arlimus force-pushed the chris-rock/solaris branch from 40a80bb to c821f75 Compare January 28, 2016 22:20
@chris-rock chris-rock added this to the 0.9.6 milestone Jan 29, 2016
srenatus added a commit that referenced this pull request Jan 29, 2016
@srenatus srenatus merged commit 41bb615 into master Jan 29, 2016
@srenatus srenatus deleted the chris-rock/solaris branch January 29, 2016 13:33
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.

4 participants