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

Unified Diff: build/config/android/rules.gni

Issue 2117153002: 🎠 Have .apk targets copy .mapping file to apks/ directory (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/android/rules.gni
diff --git a/build/config/android/rules.gni b/build/config/android/rules.gni
index 530991526caba17fbbd0a9aa896f27417acc159e..75b07e69e4d9000ec0ab4513100823f404312e9d 100644
--- a/build/config/android/rules.gni
+++ b/build/config/android/rules.gni
@@ -1872,6 +1872,19 @@ if (enable_java_templates) {
}
_dex_sources = [ _proguard_output_jar_path ]
_dex_deps = [ ":$_proguard_target" ]
+
+ _copy_proguard_mapping_target = "${_template_name}__copy_proguard_mapping"
+ copy(_copy_proguard_mapping_target) {
+ sources = [
+ "$_proguard_output_jar_path.mapping",
+ ]
+ outputs = [
+ "$_final_apk_path.mapping",
+ ]
+ deps = [
+ ":$_proguard_target",
+ ]
+ }
} else {
if (enable_multidex) {
_dex_sources = [ _jar_path ]
@@ -2159,6 +2172,13 @@ if (enable_java_templates) {
"data_deps",
])
public_deps = _final_deps
+
+ # Make the proguard .mapping file easy to find by putting it beside the .apk.
+ if (_proguard_enabled) {
+ deps = [
+ ":$_copy_proguard_mapping_target",
+ ]
+ }
}
group("${target_name}_incremental") {
forward_variables_from(invoker,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698