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

Take upstream java fixes and put them here #52

Closed
wants to merge 2 commits into from
Closed

Take upstream java fixes and put them here #52

wants to merge 2 commits into from

Conversation

petems
Copy link
Member

@petems petems commented Jan 8, 2015

Got some advice from some Puppetlabs people on how to do facts better! :)

@petems
Copy link
Member Author

petems commented Jan 8, 2015

@dgoodlad this should fix the facts and it adds in a fact for major number (ie. 7 or 8), so we can do some work toward #33 😄

@petems
Take upstream java fixes and put them here
caea0b3 
Got some advice from some Puppetlabs people on how to do facts better! :)
@petems
Improve fact to use java_home libexec
6575546 
Catches edge-case where which java is resolved, but Java is not actually installed!
@petems petems closed this May 6, 2015
@petems
Copy link
Member Author

petems commented May 6, 2015

Moved to #52

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@petems