New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 730357 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Feature



Sign in to add a comment

Allow using Retrolambda support on a per-target basis

Project Member Reported by bryanhenry@google.com, Jun 7 2017

Issue description

Currently the Retrolambda support added by  http://crbug.com/642600  can only be enabled using the global use_java8 GN arg. Add support for pre-processing a prebuilt jar with Retrolambda on a per-target basis as well.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6c9e916ccc52236c4b4cf176d0bbe1299e9de7e7

commit 6c9e916ccc52236c4b4cf176d0bbe1299e9de7e7
Author: Bryan Henry <bryanhenry@google.com>
Date: Wed Jun 07 20:34:06 2017

Add variable to enable Java 8 prebuilt processing per-target

Java 8 via Retrolambda support was added by  http://crbug.com/642600 , but
it can only be enabled globally using the 'use_java8' GN arg. This adds
a new optional 'process_java8' variable for the 'java_prebuilt' and
'android_java_prebuilt' target types to enable Retrolambda (or
equivalent, e.g. desugar) processing on a per-target basis.

BUG= 730357 

Change-Id: I75675fbc5462d5d03f5f40236c8258f1834b56b6
Reviewed-on: https://chromium-review.googlesource.com/526856
Commit-Queue: Bryan Henry <bryanhenry@google.com>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: F . <zpeng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#477752}
[modify] https://crrev.com/6c9e916ccc52236c4b4cf176d0bbe1299e9de7e7/build/config/android/internal_rules.gni
[modify] https://crrev.com/6c9e916ccc52236c4b4cf176d0bbe1299e9de7e7/build/config/android/rules.gni

Status: Verified (was: Started)

Sign in to add a comment