Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Issue 2858063002: Revert of Android: Update docs to say Java 8 is required (Closed)

Created:
3 years, 7 months ago by friedman1
Modified:
3 years, 7 months ago
Reviewers:
jbudorick, agrieve
CC:
chromium-reviews, friedman, F
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Android: Update docs to say Java 8 is required (patchset #5 id:80001 of https://codereview.chromium.org/2666783004/ ) Reason for revert: You can't do this on Trusty... java8 doesn't exist there. You have to get the debs from a later install and manually install them, which is what we have puppet doing in our environment. Original issue's description: > Android: Update docs and install-build-deps-android.sh to require java8 > > We don't strictly require java8 yet, but will soon. > > BUG=662467 > > Review-Url: https://codereview.chromium.org/2666783004 > Cr-Commit-Position: refs/heads/master@{#455459} > Committed: https://chromium.googlesource.com/chromium/src/+/8638ef810c10cee09e467b2ecd678341e12fc892 TBR=jbudorick@chromium.org,agrieve@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=662467

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+71 lines, -37 lines) Patch
M build/install-build-deps-android.sh View 4 chunks +58 lines, -37 lines 0 comments Download
M docs/android_build_instructions.md View 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
friedman1
3 years, 7 months ago (2017-05-03 19:47:12 UTC) #1
Created Revert of Android: Update docs to say Java 8 is required

Powered by Google App Engine
This is Rietveld 408576698