smola · NathanDotTo · Oct 16, 2015 · Oct 16, 2015 · Oct 16, 2015 · Oct 19, 2015
diff --git a/README.md b/README.md
@@ -14,12 +14,17 @@ None.
 Role Variables
 --------------
 
-The `java_packages` variable must be set to a list of the desired Java packages. For example:
+There are two variable for the Debian and RedHat/Fedora based distributions. See 
+below for examples:
 
 ```yaml
-java_packages:
-  - openjdk-6-jdk
-  - oracle-java7-installer
+debian_java_packages:
+  - oracle-java8-installer
+```
+
+```yaml
+redhat_java_packages:
+  - java-1.8.0-openjdk-devel
 ```
 
 # Debian / Ubuntu
@@ -46,6 +51,7 @@ Valid packages for Fedora are:
 
 - java-1.7.0-openjdk
 - java-1.8.0-openjdk
+- java-1.8.0-openjdk-devel
 
 # Others
 

diff --git a/defaults/main.yml b/defaults/main.yml
@@ -1,3 +1,4 @@
 ---
-java_packages: []
+debian_java_packages: []
+redhat_java_packages: []
 java_cleanup: True
diff --git a/tasks/Debian.yml b/tasks/Debian.yml
@@ -21,7 +21,7 @@
 
 - name: Install Java packages
   apt: pkg={{ item }} state=latest
-  with_items: java_packages
+  with_items: debian_java_packages
 
 - name: Remove unwanted Java packages
   apt: pkg={{ item }} state=absent

diff --git a/tasks/RedHat.yml b/tasks/RedHat.yml
@@ -2,7 +2,7 @@
 
 - name: Install Java packages
   yum: name={{ item }} state=latest
-  with_items: java_packages
+  with_items: redhat_java_packages
   when: ansible_os_family == 'RedHat'
 
 

diff --git a/tasks/main.yml b/tasks/main.yml
@@ -1,3 +1,5 @@
 ---
 - include: Debian.yml
   when: ansible_os_family == 'Debian'
+- include: RedHat.yml
+  when: ansible_os_family == 'RedHat'
diff --git a/tests/group_vars/test b/tests/group_vars/test
@@ -1,5 +1,5 @@
 ---
-java_packages:
+debian_java_packages:
   - oracle-java8-set-default
   - oracle-java8-installer
   - openjdk-6-jdk
diff --git a/vars/Ubuntu.yml b/vars/Ubuntu.yml
@@ -19,4 +19,4 @@ java_packages_full:
   #- openjdk-7-jre-headless:i386
   #- openjdk-7-jre-zero
   #- openjdk-7-jre-zero:i386
-java_packages_to_remove: "{{ java_packages_full | difference(java_packages) }}"
+java_packages_to_remove: "{{ java_packages_full | difference(debian_java_packages) }}"
diff --git a/vars/main.yml b/vars/main.yml
@@ -1,2 +1,2 @@
 ---
-java_needs_oracle: "{{ java_packages | join | search('oracle') }}"
+java_needs_oracle: "{{ debian_java_packages | join | search('oracle') }}"