Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Add support for Oracle Java 8#1080

Merged
hamiltont merged 9 commits intoTechEmpower:masterfrom
marko-asplund:issue-1048
Oct 17, 2014
Merged

Add support for Oracle Java 8#1080
hamiltont merged 9 commits intoTechEmpower:masterfrom
marko-asplund:issue-1048

Conversation

@marko-asplund
Copy link
Copy Markdown
Contributor

issue #1048

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add the standard RETCODE bit, so that back-to-back calls of this file don't cause fairly large delays while we re-run apt-get for no reason.

Note: Instead of return 0, you need to run update-alternatives and then return 0

@hamiltont
Copy link
Copy Markdown
Contributor

Looks exciting! I've asked for some minor changes above. In addition, could you also

  • Add a small README here explaining that Java-based framewroks can use either fw_depends java (for orackel java 7) or fw_depends java8 for oracle java 8. Explain what the binaries are called (e.g. for java 7 most people call the java binary, should they do the same for java 8 or is it named differently?)
  • Update java.sh to run update-alternatives and switch back to java-7 if necessary
  • Is there any java-based framework you feel comfortable updating to java8? That way we could see if Travis-CI green-lights these changes

I'm very glad to see this being added!

@marko-asplund
Copy link
Copy Markdown
Contributor Author

@hamiltont AFAIK, I think this PR should be ok now.

I updated servlet3-cass in my local dev env to use java8. The tests seemed to be fine, but then I ran into some problems with Travis CI (see #1082) before committing these changes. I need to figure out what's causing the CI build to fail before updating the test implementation for Java 8.

@hamiltont
Copy link
Copy Markdown
Contributor

Cool. I'm out for the next few months, have a degree to prioritize. One of
the TE team can check this for you

@marko-asplund
Copy link
Copy Markdown
Contributor Author

@hamiltont Java 8 PR updated and merged with latest upstream master branch.

@hamiltont
Copy link
Copy Markdown
Contributor

Cool, let's see what Travis has to say about it!

Note to self: see here

@marko-asplund
Copy link
Copy Markdown
Contributor Author

This PR just adds a README + Java 8 installations script, so Travis shouldn't fail due to this. While I've tested this in my development sandbox and it seems to work, no test implementations have been updated to use Java 8, yet. I'm planning to do this as part of a separate PR.

@hamiltont
Copy link
Copy Markdown
Contributor

Oh, ok. Then there's nothing to wait on for this PR I guess. Visual inspection LGTM, I'll go ahead and merge this and look forward to a PR actually using it

hamiltont added a commit that referenced this pull request Oct 17, 2014
Java: Add support for Oracle Java 8
@hamiltont hamiltont merged commit 2a7b21f into TechEmpower:master Oct 17, 2014
@marko-asplund
Copy link
Copy Markdown
Contributor Author

great, thanks!

Issue #1048 is basically a duplicate of this, so it can be closed as well.
(this one was created because I couldn't figure how to associate my PR with the existing issue)

@hamiltont hamiltont mentioned this pull request Oct 17, 2014
@hamiltont
Copy link
Copy Markdown
Contributor

@marko-asplund just to be clear, there's no reason that a framework cannot use fw_depends java java8 to install both, and then just use a full path to the binary they want, right?

Perhaps we should add that to the readme, or I predict a lot of duplicated folders (e.g. java-stuffand java-stuff-java8)

@marko-asplund
Copy link
Copy Markdown
Contributor Author

Yes, that should be totally ok.
The Java 8 installation script tries hard to retain OpenJDK 7 as the default instead of Oracle JDK 8.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants