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 uptime capability. #73 #75

Merged
merged 1 commit into from
Sep 3, 2015
Merged

Add uptime capability. #73 #75

merged 1 commit into from
Sep 3, 2015

Conversation

dbwiddis
Copy link
Member

@dbwiddis dbwiddis commented Sep 3, 2015

No description provided.

@dbwiddis dbwiddis mentioned this pull request Sep 3, 2015
@dbwiddis
Copy link
Member Author

dbwiddis commented Sep 3, 2015

Travis is failing because JAVA_HOME is not defined correctly... no idea how to fix that!

@dbwiddis
Copy link
Member Author

dbwiddis commented Sep 3, 2015

OK, seems a relatively new thing that Travis CI has implemented OS X support. Previously our builds were only running against Linux.

But OS X needs JAVA_HOME set and I don't know what to set it to for the travis VM.

We can allow errors in travis.yml on OS X for the time being, while this gets resolved...

dbwiddis added a commit that referenced this pull request Sep 3, 2015
@dbwiddis dbwiddis merged commit 3ef4b38 into oshi:master Sep 3, 2015
@dblock
Copy link
Collaborator

dblock commented Sep 3, 2015

You can set the env variables in Travis via env:, http://docs.travis-ci.com/user/environment-variables. So figure out what JAVA_HOME should be and just hard-code it in there?

@dblock
Copy link
Collaborator

dblock commented Sep 3, 2015

Note that Travis claims that JAVA_HOME is correctly set in http://docs.travis-ci.com/user/environment-variables/. Maybe it's a bug in their infrastructure?

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

2 participants