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

use Oracle Java JDK #14

Merged
merged 2 commits into from
May 31, 2017
Merged

use Oracle Java JDK #14

merged 2 commits into from
May 31, 2017

Conversation

afeld
Copy link
Contributor

@afeld afeld commented May 18, 2017

Closes #11.

Installs Oracle Java rather than OpenJDK, which is then used for running Jenkins.

$ java -version
java version "1.8.0_112"
Java(TM) SE Runtime Environment (build 1.8.0_112-b15)
Java HotSpot(TM) 64-Bit Server VM (build 25.112-b15, mixed mode)
$ ls -l /usr/java
total 4
lrwxrwxrwx. 1 root root   22 May 18 18:36 default -> /usr/java/jdk1.8.0_112
drwxr-xr-x. 9 root root 4096 May 18 18:36 jdk1.8.0_112
lrwxrwxrwx. 1 root root   22 May 18 18:36 latest -> /usr/java/jdk1.8.0_112

@afeld
Copy link
Contributor Author

afeld commented May 18, 2017

Should we be installing the Java Cryptography Extension (JCE)? It's doesn't seem to be needed by Jenkins, but probably doesn't hurt anything to install it.

@afeld
Copy link
Contributor Author

afeld commented May 24, 2017

Hold off on merging... williamyeh.oracle-java is currently broken, due to a change on the Oracle side. See https://github.com/GSA/d2d/issues/344 and William-Yeh/ansible-oracle-java#62 for more details.

@afeld afeld changed the title use Oracle Java JDK [DO NOT MERGE] use Oracle Java JDK May 24, 2017
Switch to using fork, as workaround to JDK download URL change.
@afeld afeld changed the title [DO NOT MERGE] use Oracle Java JDK use Oracle Java JDK May 30, 2017
@afeld
Copy link
Contributor Author

afeld commented May 31, 2017

@maverickquant didn't request any changes via email, so merging.

@afeld afeld merged commit c7fc943 into master May 31, 2017
@afeld afeld deleted the oracle-java branch May 31, 2017 06:25
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.

2 participants