Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Recipe doesn't install java #85

Open
drrk opened this issue May 26, 2015 · 4 comments
Open

Recipe doesn't install java #85

drrk opened this issue May 26, 2015 · 4 comments

Comments

@drrk
Copy link

drrk commented May 26, 2015

Even though the cookbook depends on the java cookbook, and the testing recipe include the java cookbook default recipe, the main cookbook doesn't install java, but depends on java to be installed.

@linc01n
Copy link
Collaborator

linc01n commented May 28, 2015

The java cookbook let you install 2 flavor of java. openjdk and the proprietary one. Which one you think we should include?

@patcon
Copy link
Collaborator

patcon commented Jun 20, 2015

Because of the wealth of config options for java, most cookbooks seem to leave it out and just document the need to add it. I would rather we did this rather than be opinionated about overrides

@bflad
Copy link
Owner

bflad commented Jul 12, 2015

Ideologically I'm in agreement with @patcon on this fortunately or unfortunately. While our testing framework will provide its own defaults to be usable in enough capacity to run the tests, I don't think we should dictate or want to wrap the Java configuration for everyone's environment. If we were to go down that route though, I'd suggest something like a "manage_java" attribute, separate java recipe, and new java attributes.

@cloneluke
Copy link

berksfile:
cookbook 'java'

recipe:

need to do java install, it was assumed in supermarket cookbook

node.default['java']['jdk_version'] = '8'
ENV['JAVA_HOME'] = '/usr/lib/jvm/java-8-openjdk-amd64'
include_recipe 'java::default'

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

No branches or pull requests

5 participants