Support Xenial, Java 8 by default, and Java 8 on Trusty #2

Merged
merged 2 commits into from May 17, 2016

Conversation

Projects
None yet
2 participants
Owner

johnsca commented May 12, 2016

No description provided.

@johnsca johnsca changed the title from Changes to support Java 8 on trusty to Support Xenial, Java 8 by default, and Java 8 on Trusty May 14, 2016

+ apt-get update -qq
+ fi
+
+ if [[ `uname -p` == 'ppc64'* ]]; then
@kwmonroe

kwmonroe May 15, 2016

Member

This looks weird as all get out to me, and I'm not near a prompt on a ppc box to verify. To earn the prestigious +1 you so desire, prove to me that the weird 'ppc64'* string does what you want it to do, that is, match ppc64le.

@johnsca

johnsca May 16, 2016

Owner

It's identical to java-installer.sh but if you want further proof:

$ test_cpu_arch="ppc64le"
$ if [[ "$test_cpu_arch" == 'ppc64'* ]]; then
>   echo "Power match"
> fi
Power match

Edit: Documented in the note under ==.

Member

kwmonroe commented May 17, 2016

Thx for the proof. LGTM.

@kwmonroe kwmonroe merged commit 0e4411a into master May 17, 2016

Member

kwmonroe commented May 18, 2016

promulgated as cs:openjdk-1

@kwmonroe kwmonroe deleted the java-8-trusty branch May 18, 2016

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